[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #15 from uros at gcc dot gnu.org --- Author: uros Date: Thu Mar 28 19:33:22 2019 New Revision: 270004 URL: https://gcc.gnu.org/viewcvs?rev=270004&root=gcc&view=rev Log: PR target/89865 * config/i386/i386.md (RMW operat

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #14 from Jakub Jelinek --- (In reply to Peter Bergner from comment #13) > (In reply to Jakub Jelinek from comment #9) > > Note, the r264897 change to the testcase was clearly bogus, because then the > > testcase is really useless, the

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 Peter Bergner changed: What|Removed |Added CC||segher at gcc dot gnu.org --- Comment #1

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #12 from Jakub Jelinek --- (In reply to Uroš Bizjak from comment #11) > (In reply to Jakub Jelinek from comment #9) > I suggest we apply my LEA patch (that is a clear improvement), and > recategorize the PR as a RA regression. Agreed

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #11 from Uroš Bizjak --- (In reply to Jakub Jelinek from comment #9) > Note, the r264897 change to the testcase was clearly bogus, because then the > testcase is really useless, the intent of the testcase was to check that all > (but

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #10 from Uroš Bizjak --- (In reply to Jakub Jelinek from comment #8) > (this is -Os, so that is what matters), r264897 made the generated code > worse, then r265398 reverted it to the previously generated code and r266385 > made it ev

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #9 from Jakub Jelinek --- Note, the r264897 change to the testcase was clearly bogus, because then the testcase is really useless, the intent of the testcase was to check that all (but the 8) peepholes did the right thing and there ar

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 Jakub Jelinek changed: What|Removed |Added CC||bergner at gcc dot gnu.org,

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #6 from Jakub Jelinek --- Ah, but that is only because r264897 adjusted the expected counts from 8 to 47/57 :(.

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #7

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #5 from Uroš Bizjak --- (In reply to Jakub Jelinek from comment #4) > I don't see the testcase FAILing on i?86 though, just on x86_64, and there > starting with Oct 2x (20th is still ok, 23rd fails, so likely r265398). The testcase i

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #4 from Jakub Jelinek --- I don't see the testcase FAILing on i?86 though, just on x86_64, and there starting with Oct 2x (20th is still ok, 23rd fails, so likely r265398). Let me have a look.

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 Uroš Bizjak changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread bernds at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 --- Comment #2 from Bernd Schmidt --- Jakub seems to be the author of gcc.target/i386/pr49095.c.

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 Richard Biener changed: What|Removed |Added Target Milestone|--- |9.0

[Bug target/89865] [9 Regression] FAIL: gcc.target/i386/pr49095.c scan-assembler-times \\\\), % 45

2019-03-28 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89865 Uroš Bizjak changed: What|Removed |Added CC||bernds at gcc dot gnu.org --- Comment #1 f