This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Test gcc.dg/vect/pr48765, run only for powerpc*-*-*, unconditionally adds -mcpu=power6 to the command line. It also adds -m64, although tests should rely on different multilib flags for that and skip the test if not lp64 if the test can only be run with -m64. For all of the Power multilib flags that I normally test, the vect test infrastructure either skips the tests (for E500) or adds "-maltivec -mcpu=970" which conflicts with the "-mcpu=power6" in this test and causes the test to fail to compile. The "-m64" definitely has to go. If the "-mcpu=power6" needs to stay in order to use the flags that triggered the ICE for which the test was added, then the test should be skipped if some other -mcpu option will override it. Another option is to remove that and figure that occasionally someone will run tests for power6. This version keeps "-mcpu=power6" and skips the tests if another -mcpu is already specified. OK for trunk, and for 4.7 when the branch opens? Janis
Attachment:
gcc-20120918-5
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |