[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #11 from Michael Meissner meissner at gcc dot gnu.org --- Author: meissner Date: Tue Mar 3 17:04:01 2015 New Revision: 221155 URL: https://gcc.gnu.org/viewcvs?rev=221155root=gccview=rev Log: 2015-03-03 Michael Meissner meiss...@linux.vnet.ibm.com PR 65138/target * config/rs6000/rs6000-tables.opt: Regenerate table. Modified: trunk/gcc/ChangeLog trunk/gcc/config/rs6000/rs6000-tables.opt
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 Aldy Hernandez aldyh at gcc dot gnu.org changed: What|Removed |Added CC||aldyh at gcc dot gnu.org --- Comment #12 from Aldy Hernandez aldyh at gcc dot gnu.org --- Is this fixed; can we close it?
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #13 from Michael Meissner meissner at gcc dot gnu.org --- I'm preparing the patches for the back ports to GCC 4.8/4.9. Once those are committed, I will close the PR.
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 Michael Meissner meissner at gcc dot gnu.org changed: What|Removed |Added Attachment #34923|0 |1 is obsolete|| Attachment #34925|0 |1 is obsolete|| --- Comment #9 from Michael Meissner meissner at gcc dot gnu.org --- Created attachment 34926 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34926action=edit Revised patch #3 This patch changes the logic to use TARGET_DEFAULT. If TARGET_DEFAULT is 0, it falls back to the defaults for powerpc, powerpc64, or powerpc64le.
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 Michael Meissner meissner at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at gcc dot gnu.org |meissner at gcc dot gnu.org
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #8 from Michael Meissner meissner at gcc dot gnu.org --- Created attachment 34925 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34925action=edit Revised patch #2
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #10 from Michael Meissner meissner at gcc dot gnu.org --- Author: meissner Date: Tue Mar 3 00:59:56 2015 New Revision: 221132 URL: https://gcc.gnu.org/viewcvs?rev=221132root=gccview=rev Log: 2015-03-02 Michael Meissner meiss...@linux.vnet.ibm.com PR 65138/target * config/rs6000/rs6000-cpus.def (powerpc64le): Add new generic processor type for 64-bit little endian PowerPC. * config/rs6000/rs6000.c (rs6000_option_override_internal): If -mdebug=reg, print TARGET_DEFAULT. Fix logic to use TARGET_DEFAULT if there is no default cpu. Fix -mdebug=reg printing built-in mask so it does not pass NULL pointers. * doc/invoke.texi (IBM RS/6000 and PowerPC options): Document -mcpu=powerpc64le. Modified: trunk/gcc/ChangeLog trunk/gcc/config/rs6000/rs6000-cpus.def trunk/gcc/config/rs6000/rs6000.c trunk/gcc/doc/invoke.texi
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #6 from Michael Meissner meissner at gcc dot gnu.org --- Created attachment 34923 -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34923action=edit Proposed patch to fix the problem I'm going to run full tests on this patch, but this patch appears to fix the problem.
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #7 from David Edelsohn dje at gcc dot gnu.org --- Allowing users to specify -mcpu=powerpc64le is all well and good, but if TARGET_DEFAULT is set, it should not be gratuitously overridden in rs6000_option_override. The logic is not correct and defaults should not be inherited from the processor name.
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #5 from Michael Meissner meissner at gcc dot gnu.org --- Ok, after figuring out that I was building on gcc2-power8.osuosl.org (the old big endian system that is going away), I checked out the current revision (221024) and rebuilt a tool chain on gcc2-2-power8.osuosl.org. I now see the error if I build without the --with-cpu=power8 option (the options I used were --prefix=xxx and --disable-multilib).
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 Michael Meissner meissner at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2015-02-26 Ever confirmed|0 |1
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #3 from Michael Meissner meissner at gcc dot gnu.org --- I just built a toolchain on gcc2-power8.osuosl.org with the configuration line: /home/meissner/fsf-src/trunk/configure --prefix=/home/meissner/fsf-install-ppcle64/trunk --enable-languages=c,c++,ada,fortran,java,objc,obj-c++ --with-long-double-128 --enable-secureplt --enable-__cxa_atexit --disable-libsanitizer --with-cpu=power8 And after doing a make check, install, and check, I did not see any of these tests generate errors with -flto -fuse-linker-plugin -flto-partition=none. What was the exact command line you used to configure the compiler, and what was your path?
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 --- Comment #4 from Markus Trippelsdorf trippels at gcc dot gnu.org --- (In reply to Michael Meissner from comment #3) I just built a toolchain on gcc2-power8.osuosl.org with the configuration line: /home/meissner/fsf-src/trunk/configure --prefix=/home/meissner/fsf-install-ppcle64/trunk --enable-languages=c,c++,ada,fortran,java,objc,obj-c++ --with-long-double-128 --enable-secureplt --enable-__cxa_atexit --disable-libsanitizer --with-cpu=power8 And after doing a make check, install, and check, I did not see any of these tests generate errors with -flto -fuse-linker-plugin -flto-partition=none. What was the exact command line you used to configure the compiler, and what was your path? It doesn't happen --with-cpu=power8. So please leave that out. I was using: ../gcc/configure --disable-bootstrap --disable-libstdcxx-pch --disable-libvtv --disable-libitm --disable-libcilkrts --disable-libssp --disable-libgomp --disable-werror --disable-multilib --enable-languages=c,c++
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added CC||dje at gcc dot gnu.org, ||jakub at gcc dot gnu.org --- Comment #2 from Jakub Jelinek jakub at gcc dot gnu.org --- Perhaps something is broken in the rs6000 target attribute support (which LTO now uses for all functions)? Like, PR61925 fix not ported to rs6000 yet?
[Bug target/65138] [5 Regression] testsuite ICEs on powerpc64le
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65138 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added Target Milestone|--- |5.0 --- Comment #1 from Richard Biener rguenth at gcc dot gnu.org --- Only with -flto? huh.