[PATCH] rs6000: Fix 2 for PR92661, Do not define builtins that overload disabled builtins
Peter Bergner
bergner@linux.ibm.com
Wed Dec 18 15:31:00 GMT 2019
On 12/18/19 8:15 AM, Segher Boessenkool wrote:
>> +/* { dg-do compile { target { powerpc*-*-* } } } */
>> +/* { dg-options "-w -O2 -mdejagnu-cpu=power9" } */
>
> You don't need that target clause in gcc.target/powerpc (and dg-do compile
> is the default, but having it explicit is also fine of course).
I think leaving the bare dg-do compile (ie, no target) is nice,
for newbies who don't know that dg-do compile is the default.
>> --- gcc/testsuite/gcc.target/powerpc/dfp-dd.c (revision 278980)
>> +++ gcc/testsuite/gcc.target/powerpc/dfp-dd.c (working copy)
>> @@ -1,6 +1,7 @@
>> /* Test generation of DFP instructions for POWER6. */
>> /* Origin: Janis Johnson <janis187@us.ibm.com> */
>> -/* { dg-do compile { target { powerpc*-*-linux* && powerpc_fprs } } } */
>> +/* { dg-do compile { target { powerpc*-*-linux* } } } */
>> +/* { dg-require-effective-target dfp_hw } */
>
> You can remove powerpc_fprs now because it became redundant? Cool.
Right, hard dfp support requires we have hard float support.
> But dfp_hw is the wrong conditions for a dg-do compile test.
Ok, yes. Looking closer, that dfp_hw is a runtime test and not
what we want. I'll change this to using "hard_dfp" which is a
compile time test.
> Nice cleanups! Please fix that dfp_hw thing, and then, okay for trunk,
> Thanks!
Will do, thanks. I'll commit this after making these changes and
rerunning the updated test cases.
Peter
More information about the Gcc-patches
mailing list