https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
Martin Liška changed:
What|Removed |Added
CC||mcn at onet dot pl
--- Comment #17 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
Martin Liška changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #15 from CVS Commits ---
The master branch has been updated by Martin Liska :
https://gcc.gnu.org/g:8e836af61b7027c0819da62c12a8d18b7c46f3fc
commit r12-5824-g8e836af61b7027c0819da62c12a8d18b7c46f3fc
Author: Martin Liska
Date: Mo
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #14 from Nils Smeds ---
OK, the wording "aggressive" makes it look counter-intuitive that it is set
at -O0, but apparently it is correct.
/Nils
On Thu, Dec 2, 2021 at 3:02 PM marxin at gcc dot gnu.org <
gcc-bugzi...@gcc.gnu.org> wr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #13 from Nils Smeds ---
OK, then it makes sense
On Thu, Dec 2, 2021 at 3:02 PM marxin at gcc dot gnu.org <
gcc-bugzi...@gcc.gnu.org> wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
>
> --- Comment #12 from Martin Liška
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #12 from Martin Liška ---
> I notice that you don't see -faggressive-loop-optimizations in your diff
> output of the patch.
Correct. The option should be enabled at all levels:
faggressive-loop-optimizations
Common Var(flag_aggress
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #11 from Nils Smeds ---
I also noticed the following with this patch (if I applied it correctly)
[tooladmin@login2302 gcc-11.2.0]$ gcc -O0 -Q --help=optimize t.c | grep agg
-faggressive-loop-optimizations [enabled]
[tooladm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #10 from Nils Smeds ---
Is there a benefit to augment the -Q output as follows?
[tooladmin@login2302 build]$ diff -r -p a b
diff -r -p a/gcc/opts.c b/gcc/opts.c
*** a/gcc/opts.cWed Jul 28 08:55:07 2021
--- b/gcc/opts.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
Martin Liška changed:
What|Removed |Added
Keywords||patch
--- Comment #9 from Martin Liška
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #8 from Nils Smeds ---
(In reply to Nils Smeds from comment #7)
> (In reply to Martin Liška from comment #6)
> > Lemme take a look.
>
> Great, thanks.
prefetch-loop-arrays appears to be activated on arm,s390 and i386 at level -O3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #7 from Nils Smeds ---
(In reply to Martin Liška from comment #6)
> Lemme take a look.
Great, thanks.
What happens in real life inside the code is beyond me, I am mostly after that
the documentation should describe what is really
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
Richard Biener changed:
What|Removed |Added
CC||marxin at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #4 from Nils Smeds ---
(In reply to Andrew Pinski from comment #1)
> -fprefetch-loop-arrays is not enabled by default on all targets.
$ gcc --version
gcc (GCC) 11.2.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free so
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #3 from Andrew Pinski ---
Oh the "enabled" thing when doing --help is confusing in the case of
-fprefetch-loop-arrays:
fprefetch-loop-arrays
Common Var(flag_prefetch_loop_arrays) Init(-1) Optimization
Generate prefetch instructions,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #2 from Jonathan Wakely ---
Thanks for the report, and the patch. After addressing Andrew's comment could
you please send a new patch to the gcc-patches mailing list for review? Patches
attached here tend to get missed or forgotten a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103438
--- Comment #1 from Andrew Pinski ---
-fprefetch-loop-arrays is not enabled by default on all targets.
17 matches
Mail list logo