diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2018-04-24 18:42:59 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2018-04-24 18:42:59 +0200 |
| commit | e8ac3359dc7c105b7f1edcf866517d3d273303c5 (patch) | |
| tree | 1ff19017d859b10e48e58e07142886f6695875e2 | |
| parent | 8624f8685dd6538583e310db0fa4dc30e2f7d5bc (diff) | |
| download | gcc-e8ac3359dc7c105b7f1edcf866517d3d273303c5.zip gcc-e8ac3359dc7c105b7f1edcf866517d3d273303c5.tar.gz gcc-e8ac3359dc7c105b7f1edcf866517d3d273303c5.tar.bz2 | |
re PR target/85511 ([X86] Using __builtin_ia32_writeeflags_u32 in 64-bit mode causes internal compiler error)
PR target/85511
* config/i386/i386.c (ix86_init_mmx_sse_builtins): Don't define
__builtin_ia32_readeflags_u32 and __builtin_ia32_writeeflags_u32
if TARGET_64BIT.
* gcc.target/i386/pr85511.c: New test.
From-SVN: r259609
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/config/i386/i386.c | 22 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 3 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.target/i386/pr85511.c | 15 |
4 files changed, 37 insertions, 8 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e43a931..2992064 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,10 @@ 2018-04-24 Jakub Jelinek <jakub@redhat.com> + PR target/85511 + * config/i386/i386.c (ix86_init_mmx_sse_builtins): Don't define + __builtin_ia32_readeflags_u32 and __builtin_ia32_writeeflags_u32 + if TARGET_64BIT. + PR target/85503 * config/rs6000/rs6000-p8swap.c (const_load_sequence_p): Punt if const_vector is not CONST_VECTOR or SYMBOL_REF for a constant pool diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index fada22b..a3c3870 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -31934,14 +31934,20 @@ ix86_init_mmx_sse_builtins (void) IX86_BUILTIN_SBB64); /* Read/write FLAGS. */ - def_builtin (0, "__builtin_ia32_readeflags_u32", - UNSIGNED_FTYPE_VOID, IX86_BUILTIN_READ_FLAGS); - def_builtin (OPTION_MASK_ISA_64BIT, "__builtin_ia32_readeflags_u64", - UINT64_FTYPE_VOID, IX86_BUILTIN_READ_FLAGS); - def_builtin (0, "__builtin_ia32_writeeflags_u32", - VOID_FTYPE_UNSIGNED, IX86_BUILTIN_WRITE_FLAGS); - def_builtin (OPTION_MASK_ISA_64BIT, "__builtin_ia32_writeeflags_u64", - VOID_FTYPE_UINT64, IX86_BUILTIN_WRITE_FLAGS); + if (TARGET_64BIT) + { + def_builtin (OPTION_MASK_ISA_64BIT, "__builtin_ia32_readeflags_u64", + UINT64_FTYPE_VOID, IX86_BUILTIN_READ_FLAGS); + def_builtin (OPTION_MASK_ISA_64BIT, "__builtin_ia32_writeeflags_u64", + VOID_FTYPE_UINT64, IX86_BUILTIN_WRITE_FLAGS); + } + else + { + def_builtin (0, "__builtin_ia32_readeflags_u32", + UNSIGNED_FTYPE_VOID, IX86_BUILTIN_READ_FLAGS); + def_builtin (0, "__builtin_ia32_writeeflags_u32", + VOID_FTYPE_UNSIGNED, IX86_BUILTIN_WRITE_FLAGS); + } /* CLFLUSHOPT. */ def_builtin (OPTION_MASK_ISA_CLFLUSHOPT, "__builtin_ia32_clflushopt", diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 0c1c0be..12fd465 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,8 @@ 2018-04-24 Jakub Jelinek <jakub@redhat.com> + PR target/85511 + * gcc.target/i386/pr85511.c: New test. + PR target/85503 * g++.dg/ext/pr85503.C: New test. diff --git a/gcc/testsuite/gcc.target/i386/pr85511.c b/gcc/testsuite/gcc.target/i386/pr85511.c new file mode 100644 index 0000000..a082a5e --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr85511.c @@ -0,0 +1,15 @@ +/* PR target/85511 */ +/* { dg-do compile } */ +/* { dg-options "-Wimplicit-function-declaration" } */ + +unsigned int +foo (void) +{ + return __builtin_ia32_readeflags_u32 (); /* { dg-warning "implicit declaration of function" "" { target { ! ia32 } } } */ +} + +void +bar (unsigned int x) +{ + __builtin_ia32_writeeflags_u32 (x); /* { dg-warning "implicit declaration of function" "" { target { ! ia32 } } } */ +} |