[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142 --- Comment #19 from Segher Boessenkool segher at gcc dot gnu.org 2012-08-16 00:57:44 UTC --- Author: segher Date: Thu Aug 16 00:57:37 2012 New Revision: 190427 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=190427 Log: 2012-08-15 Segher Boessenkool seg...@kernel.crashing.org gcc/ PR target/54142 * config/rs6000/driver-rs6000.c (asm_names): Use %(asm_default) instead of -mcom. * config/rs6000/rs6000.h (ASM_CPU_SPEC): Ditto. Modified: trunk/gcc/ChangeLog trunk/gcc/config/rs6000/driver-rs6000.c trunk/gcc/config/rs6000/rs6000.h
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142 Segher Boessenkool segher at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED --- Comment #20 from Segher Boessenkool segher at gcc dot gnu.org 2012-08-16 02:17:30 UTC --- Fixed.
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #14 from Jakub Jelinek jakub at gcc dot gnu.org 2012-08-14
07:49:11 UTC ---
Yeah, IMHO it should have added
%{!mpower*: %(asm_default)}} \
line instead of
%{!mpowerpc*: -mcom}} \
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142
--- Comment #15 from Gary Funck gary at intrepid dot com 2012-08-14 13:17:26
UTC ---
(In reply to comment #14)
Yeah, IMHO it should have added
%{!mpower*: %(asm_default)}} \
line instead of
%{!mpowerpc*: -mcom}} \
That change fixed the build failure on a POWER7 system running RHEL 6.2 that I
tried it on, where it failed without the patch.
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142 Segher Boessenkool segher at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED CC||segher at gcc dot gnu.org AssignedTo|unassigned at gcc dot |segher at gcc dot gnu.org |gnu.org | --- Comment #16 from Segher Boessenkool segher at gcc dot gnu.org 2012-08-14 16:44:09 UTC --- Mine.
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142 David Edelsohn dje at gcc dot gnu.org changed: What|Removed |Added CC||dje at gcc dot gnu.org --- Comment #17 from David Edelsohn dje at gcc dot gnu.org 2012-08-14 17:09:57 UTC --- Jakub's fix looks right.
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142
--- Comment #18 from Segher Boessenkool segher at gcc dot gnu.org 2012-08-14
22:54:35 UTC ---
Right, but
%{!mpowerpc*: %(asm_default)}} \
instead, since the -mpower option is no more.
I didn't even pick the wrong condition branch there: the original
code does not do the right thing for gcc -mno-power, which is
effectively always on after my patch. But let's not always do
the wrong thing; patch coming up (it needs changing in a few places
more).
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142 Andreas Schwab sch...@linux-m68k.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2012-08-13 Target Milestone|--- |4.8.0 Summary|ppc64 build failure - |[4.8 regression] ppc64 |Unrecognized opcode: `sldi' |build failure - |(and `srdi`)|Unrecognized opcode: `sldi' ||(and `srdi`) Ever Confirmed|0 |1 --- Comment #12 from Andreas Schwab sch...@linux-m68k.org 2012-08-13 23:14:04 UTC --- Confirmed.
[Bug target/54142] [4.8 regression] ppc64 build failure - Unrecognized opcode: `sldi' (and `srdi`)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54142 --- Comment #13 from Paul H. Hargrove PHHargrove at lbl dot gov 2012-08-14 00:01:08 UTC --- (In reply to comment #9) Of course, if 'sldi' and 'slri' ARE supposed to be supported in common mode, then this is a binutils bug. I've confirmed that these instruction are NOT in the POWER ISA, and thus gas is correct in rejecting them in -mcom mode. Reference: Appendix F. New Instructions in Book I: PowerPC User Instruction Set Architecture which is one volume in PowerPC Architecture Book, Version 2.02 PDF: http://download.boulder.ibm.com/ibmdl/pub/software/dw/library/es-ppcbook1.zip Specifically, the Appendix F table lists instructions in the PowerPC ISA that are not part of the POWER ISA. The rldicl and rldicr instructions are among those listed (and as noted previously, sldi and srdi are just extended mnemonics for those two).
