* [PATCH] Typo in comment in tree-ssa-loop-prefetch
@ 2010-09-01 14:55 Ian Bolton
2010-09-01 20:31 ` Zdenek Dvorak
0 siblings, 1 reply; 2+ messages in thread
From: Ian Bolton @ 2010-09-01 14:55 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 801 bytes --]
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
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.
[-- Attachment #2: Typo.diff --]
[-- Type: application/octet-stream, Size: 620 bytes --]
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.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Typo in comment in tree-ssa-loop-prefetch
2010-09-01 14:55 [PATCH] Typo in comment in tree-ssa-loop-prefetch Ian Bolton
@ 2010-09-01 20:31 ` Zdenek Dvorak
0 siblings, 0 replies; 2+ messages in thread
From: Zdenek Dvorak @ 2010-09-01 20:31 UTC (permalink / raw)
To: Ian Bolton; +Cc: gcc-patches
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.
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-09-01 20:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-01 14:55 [PATCH] Typo in comment in tree-ssa-loop-prefetch Ian Bolton
2010-09-01 20:31 ` Zdenek Dvorak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).