[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 Alan Modra amodra at gmail dot com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 --- Comment #5 from Alan Modra amodra at gcc dot gnu.org --- Author: amodra Date: Mon Mar 16 03:29:36 2015 New Revision: 221445 URL: https://gcc.gnu.org/viewcvs?rev=221445root=gccview=rev Log: PR target/63150 gcc/ * config/rs6000/rs6000.md (bswapdi2): Remove one scratch reg. Modify Z-r bswapdi splitter to use dest in place of scratch. In r-Z and Z-r bswapdi splitter rename word_high, word_low to word1, word2 and rearrange logic to suit. (bswapdi2_64bit): Remove early clobber on Z-r alternative. (bswapdi2_ldbrx): Likewise. Remove '??' on r-r. (bswapdi2_32bit): Remove early clobber on Z-r alternative. Add one '?' on r-r. Modify Z-r splitter to avoid need for early clobber. gcc/testsuite/ * gcc.target/powerpc/pr53199.c: Add extra functions. Revert 2014-12-05 change. Modified: trunk/gcc/ChangeLog trunk/gcc/config/rs6000/rs6000.md trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/gcc.target/powerpc/pr53199.c
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 --- Comment #4 from Martin Sebor msebor at gcc dot gnu.org --- Created attachment 34812 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34812action=edit Patch referenced in comment #2 with conflicts resolved and test for pr53199 re-enabled. The test no longer fails (in the reporter's configuration) due to the commit below. Backing it out causes the test to fail again with today's trunk. After applying the patch referenced in comment 2 (and resolving the two conflicts) and backing out the change below, the test passes again. The modified patch is attached. commit 7d6914c8d62668f03687d4d28630a1946ee2052c Author: meissner meissner@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Fri Dec 5 17:50:40 2014 + 2014-12-05 Michael Meissner meiss...@linux.vnet.ibm.com PR target/53199 * gcc.target/powerpc/pr53199.c: Limit tests to 64-bit. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@218431 138bc75d-0d04-0410-961f-82ee72b054a4
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Keywords||missed-optimization Priority|P3 |P2
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added Target Milestone|4.9.2 |4.9.3 --- Comment #3 from Jakub Jelinek jakub at gcc dot gnu.org --- GCC 4.9.2 has been released.
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Known to work||4.8.3 Target Milestone|--- |4.9.2 Summary|[4.9 regression] FAIL: |[4.9/5 regression] FAIL: |gcc.target/powerpc/pr53199. |gcc.target/powerpc/pr53199. |c scan-assembler-times *|c scan-assembler-times *
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 Segher Boessenkool segher at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2014-09-03 CC||segher at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Segher Boessenkool segher at gcc dot gnu.org --- Confirmed. Happens with -m32 -mno-lra only. bswapdi2_32bit only allows Z for memory operands; reload decides to use a register.
[Bug regression/63150] [4.9/5 regression] FAIL: gcc.target/powerpc/pr53199.c scan-assembler-times *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63150 Pat Haugen pthaugen at gcc dot gnu.org changed: What|Removed |Added CC||pthaugen at gcc dot gnu.org --- Comment #2 from Pat Haugen pthaugen at gcc dot gnu.org --- Alan posted a patch here: http://gcc.gnu.org/ml/gcc-patches/2013-12/msg01487.html, but thread seems to have went stale.
