https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45223
--- Comment #9 from Richard Biener ---
But this but is about non-MEMs while the fixes were involving only memory ops
IIRC.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45223
Andrew Pinski changed:
What|Removed |Added
See Also||https://gcc.gnu.org/bugzill
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45223
Bug 45223 depends on bug 42108, which changed state.
Bug 42108 Summary: [4.9 Regression] 50% performance regression
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42108
What|Removed |Added
--- Comment #7 from ubizjak at gmail dot com 2010-09-04 10:03 ---
Unassigning...
--
ubizjak at gmail dot com changed:
What|Removed |Added
AssignedTo|ubizjak at gm
--- Comment #6 from ubizjak at gmail dot com 2010-08-12 14:46 ---
Ouch, "Here are the ration of before and after on Intel Core i7. Gzip slowed
down by 10 to 20%." [1]
Richi says: "The fix is to teach LIM to do conditional invariant motion."
Probably also related to PR42108 catch-all PR
--- Comment #5 from ubizjak at gmail dot com 2010-08-07 15:27 ---
Patch at [1].
[1] http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00553.html
--
ubizjak at gmail dot com changed:
What|Removed |Added
--
--- Comment #4 from ubizjak at gmail dot com 2010-08-07 11:26 ---
(In reply to comment #3)
> Patch of comment #1 loops obviously OK to me. We shouldn't want to move
> trapping insns in any case that I can think of.
OK, will post patch to gcc-patches after regression test.
--
ubizjak
--- Comment #3 from steven at gcc dot gnu dot org 2010-08-07 10:57 ---
Patch of comment #1 loops obviously OK to me. We shouldn't want to move
trapping insns in any case that I can think of.
--
steven at gcc dot gnu dot org changed:
What|Removed |A
--- Comment #2 from ubizjak at gmail dot com 2010-08-07 08:40 ---
Ugh, with a bit changed testcase:
--cut here--
extern void exit (int);
extern void abort (void);
volatile float a = 1;
volatile float b = 0;
volatile int x = 2;
volatile signed int r = 8;
void __attribute__((noinline))
--- Comment #1 from ubizjak at gmail dot com 2010-08-07 08:33 ---
Following patch fixes this problem:
Index: gcse.c
===
--- gcse.c (revision 162975)
+++ gcse.c (working copy)
@@ -1693,7 +1693,7 @@ compute_hash_tab
10 matches
Mail list logo