diff mbox

[rs6000] Fix PR target/80246, DFP builtins using the wrong types

Message ID fd2aa73b-72c2-2dc7-1285-b0b3672222ed@vnet.ibm.com
State New
Headers show

Commit Message

Peter Bergner April 3, 2017, 4:04 p.m. UTC 
On 4/2/17 1:53 PM, Segher Boessenkool wrote:
> I also have a fix for the dfp-builtin-1.c problem.

Something like the following maybe?  It seems to work for me.
I think the hard_dfp predicate was added after the dfp-builtin-[12].c
test cases were added, which is maybe why Mike used powerpc_vsx_ok?
Seems like we should switch it to using hard_dfp now.

Peter
diff mbox

Patch

Index: dfp-builtin-1.c
===================================================================
--- dfp-builtin-1.c	(revision 246648)
+++ dfp-builtin-1.c	(working copy)
@@ -1,5 +1,5 @@ 
 /* { dg-do compile { target { powerpc*-*-linux* } } } */
-/* { dg-require-effective-target powerpc_vsx_ok } */
+/* { dg-require-effective-target hard_dfp } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { "-mcpu=power7" } } */
 /* { dg-options "-mcpu=power7 -O2" } */
 /* { dg-final { scan-assembler-times "ddedpd " 4    } } */
@@ -8,8 +8,10 @@ 
 /* { dg-final { scan-assembler-times "diex "   1    } } */
 /* { dg-final { scan-assembler-times "dscli "  2    } } */
 /* { dg-final { scan-assembler-times "dscri "  2    } } */
-/* { dg-final { scan-assembler-times "std "    1    } } */
-/* { dg-final { scan-assembler-times "ld "     1    } } */
+/* { dg-final { scan-assembler-times "std "    1    { target lp64 } } } */
+/* { dg-final { scan-assembler-times "ld "     1    { target lp64 } } } */
+/* { dg-final { scan-assembler-times "stw "    2    { target ilp32 } } } */
+/* { dg-final { scan-assembler-times "lwz "    2    { target ilp32 } } } */
 /* { dg-final { scan-assembler-times "stfd "   1    } } */
 /* { dg-final { scan-assembler-times "lfd "    1    } } */
 /* { dg-final { scan-assembler-not   "bl __builtin" } } */
Index: dfp-builtin-2.c
===================================================================
--- dfp-builtin-2.c	(revision 246648)
+++ dfp-builtin-2.c	(working copy)
@@ -1,7 +1,6 @@ 
 /* { dg-do compile { target { powerpc*-*-linux* } } } */
-/* { dg-skip-if "" { powerpc*-*-darwin* } { "*" } { "" } } */
-/* { dg-skip-if "" { powerpc*-*-*spe* } { "*" } { "" } } */
 /* { dg-require-effective-target powerpc_vsx_ok } */
+/* { dg-require-effective-target hard_dfp } */
 /* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { "-mcpu=power7" } } */
 /* { dg-options "-mcpu=power7 -O2" } */
 /* { dg-final { scan-assembler-times "ddedpdq " 4    } } */