[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-10 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #15 from Richard Biener --- So the issue is really that for for (int i = 0; i < n; ++i) { double tem1 = a4[i*4] + a4[i*4+n*4] (**); double tem2 = a4[i*4+2*n*4+1]; a4[i*4+n*4+1] = tem1; a4[i*4+1] = tem2;

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-10 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #14 from Richard

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-10 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #13 from Martin Liška --- Can be reproduced also on e.g. a Haswell machine: -Ofast -march=haswell -g -funroll-loops

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-10 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #12 from Martin Liška --- Theoretically similar to PR87214, but the patch was backported and this issues is present in 8.3.1.

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-10 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #11 from Martin Liška --- Created attachment 46124 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46124=edit Vectorizer and optimized dumps So I can confirm the problematic file is mapz_module.fppized.f90. The problematic

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #10 from Richard Biener --- Looking at the rev. and the context I figured the original caller was added for a case that can no longer happen (SAME_DR_STMT set, that can never happen since we rewrote interleaving chain detection for

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #9 from Martin Liška --- However, '--size=test' helps here, fails quickly. With the revision, there 2 files are difference: mapz_module.fppized.o.s and optics_lib.o.s. I suspect the later one.

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 Martin Liška changed: What|Removed |Added Status|WAITING |NEW

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #8 from Martin Liška --- > > Please use GCC 8 branch, not trunk. The problem only shows up on GCC 8 > branch. I can confirm that with r265453 I see: *** Miscompare of cam4_validate.txt; for details see

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 Martin Liška changed: What|Removed |Added CC||marxin at gcc dot gnu.org --- Comment #3

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-08 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 --- Comment #2 from H.J. Lu --- On trunk, r265457 fixed 527.cam4_r in SPEC CPU 2017 with: -march=native -Ofast -funroll-loops

[Bug tree-optimization/90018] [8 Regression] r265453 miscompiled 527.cam4_r in SPEC CPU 2017

2019-04-08 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90018 H.J. Lu changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|