Hi!

I've noticed this testcase FAILs, there are many reasons for that:
1) hardcoded use of -m32, which isn't really available on powerpc64le
2) -c in dg-options
3) use of target_clones attribute without ppc_cpu_supports_hw effective
   target
Also, there is no need to use both -mcpu=power7 -mcpu=e300c3, either of
those is enough to get the ICE if the ira-colors.c change is reverted.
For 1), while the testcase doesn't ICE unless ilp32 with the fix reverted,
it compiles just fine with current trunk, so I see no reason to limit it to
ilp32 only.

Regtested on powerpc64le-linux, ok for trunk?

2019-01-08  Jakub Jelinek  <ja...@redhat.com>

        PR target/88457
        * gcc.target/powerpc/pr88457.c: Remove -m32, -c and -mcpu=e300c3 from
        dg-options.  Require ppc_cpu_supports_hw effective target instead of
        powerpc64*-*-*.

--- gcc/testsuite/gcc.target/powerpc/pr88457.c.jj       2018-12-21 
00:40:46.882973124 +0100
+++ gcc/testsuite/gcc.target/powerpc/pr88457.c  2019-01-08 18:46:13.633442809 
+0100
@@ -1,5 +1,6 @@
-/* { dg-do compile  { target { powerpc64*-*-* } } } */
-/* { dg-options "-m32 -mcpu=power7 -O1 -fexpensive-optimizations --param 
ira-max-conflict-table-size=0 --param max-cse-insns=3 -c -mcpu=e300c3" } */
+/* { dg-do compile } */
+/* { dg-require-effective-target ppc_cpu_supports_hw } */
+/* { dg-options "-mcpu=power7 -O1 -fexpensive-optimizations --param 
ira-max-conflict-table-size=0 --param max-cse-insns=3" } */
 
 __attribute__((target_clones("cpu=power9,default")))
 long mod_func (long a, long b)

        Jakub

Reply via email to