Patchwork Typo in comment in tree-ssa-loop-prefetch

login
register
mail settings
Submitter Ian Bolton
Date Sept. 1, 2010, 2:43 p.m.
Message ID <000801cb49e3$fa74e600$ef5eb200$@bolton@arm.com>
Download mbox | patch
Permalink /patch/63381/
State New
Headers show

Comments

Ian Bolton - Sept. 1, 2010, 2:43 p.m.
Hi there,

I believe the attached patch (also pasted below for ease of review)
achieves the intended meaning of the comment at the top of
tree-ssa-loop-prefetch.c.

OK to commit?

Cheers,
Ian
Zdenek Dvorak - Sept. 1, 2010, 8:05 p.m.
Hi,

> I believe the attached patch (also pasted below for ease of review)
> achieves the intended meaning of the comment at the top of
> tree-ssa-loop-prefetch.c.
> 
> OK to commit?

yes,

Zdenek

> Cheers,
> Ian
> 
> 
> Index: gcc/tree-ssa-loop-prefetch.c
> ===================================================================
> --- gcc/tree-ssa-loop-prefetch.c	(revision 163732)
> +++ gcc/tree-ssa-loop-prefetch.c	(working copy)
> @@ -80,7 +80,7 @@ along with GCC; see the file COPYING3.  
>         (2) has PREFETCH_MOD 64
>         (3) has PREFETCH_MOD 4
>         (4) has PREFETCH_MOD 1.  We do not set PREFETCH_BEFORE here, since
> -           the cache line accessed by (4) is the same with probability only
> +           the cache line accessed by (5) is the same with probability only
>  	   7/32.
>         (5) has PREFETCH_MOD 1 as well.
> 
>

Patch

Index: gcc/tree-ssa-loop-prefetch.c
===================================================================
--- gcc/tree-ssa-loop-prefetch.c	(revision 163732)
+++ gcc/tree-ssa-loop-prefetch.c	(working copy)
@@ -80,7 +80,7 @@  along with GCC; see the file COPYING3.  
        (2) has PREFETCH_MOD 64
        (3) has PREFETCH_MOD 4
        (4) has PREFETCH_MOD 1.  We do not set PREFETCH_BEFORE here, since
-           the cache line accessed by (4) is the same with probability only
+           the cache line accessed by (5) is the same with probability only
 	   7/32.
        (5) has PREFETCH_MOD 1 as well.