diff options
author | Richard Sandiford <richard.sandiford@arm.com> | 2019-08-21 13:59:31 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2019-08-21 13:59:31 +0000 |
commit | 077eb7af6e296e45e8f333f799170685b4d77b03 (patch) | |
tree | d46a050af0f61c366729b5daee6cc2e9cf875fea /gcc | |
parent | d6dea10acfd9d775f260a2e7c319bb1ee64c0af0 (diff) | |
download | gcc-077eb7af6e296e45e8f333f799170685b4d77b03.zip gcc-077eb7af6e296e45e8f333f799170685b4d77b03.tar.gz gcc-077eb7af6e296e45e8f333f799170685b4d77b03.tar.bz2 |
[C++] Protect call to copy_attributes_to_builtin (PR91505)
copy_attributes_to_builtin only handles BUILT_IN_NORMAL, but C++ was
calling it immediately after the:
if (DECL_BUILT_IN_CLASS (newdecl) == BUILT_IN_NORMAL)
block. The corresponding C code calls it inside the block instead.
2019-08-21 Richard Sandiford <richard.sandiford@arm.com>
gcc/cp/
PR c++/91505
* decl.c (duplicate_decls): Call copy_attributes_to_builtin inside
the BUILT_IN_NORMAL block rather than afterward.
gcc/testsuite/
PR c++/91505
* g++.target/i386/crc32-4.C: New test.
From-SVN: r274799
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/cp/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/cp/decl.c | 4 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/g++.target/i386/crc32-4.C | 4 |
4 files changed, 17 insertions, 2 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index c40bc9c..d47c2f7 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2019-08-21 Richard Sandiford <richard.sandiford@arm.com> + + PR c++/91505 + * decl.c (duplicate_decls): Call copy_attributes_to_builtin inside + the BUILT_IN_NORMAL block rather than afterward. + 2019-08-19 Marek Polacek <polacek@redhat.com> PR c++/91264 - detect modifying const objects in constexpr. diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index 08b7baa..88aa69c 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -2565,9 +2565,9 @@ duplicate_decls (tree newdecl, tree olddecl, bool newdecl_is_friend) set_builtin_decl_declared_p (fncode, true); break; } - } - copy_attributes_to_builtin (newdecl); + copy_attributes_to_builtin (newdecl); + } } if (new_defines_function) /* If defining a function declared with other language diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index e8d22d1..27c5144 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2019-08-21 Richard Sandiford <richard.sandiford@arm.com> + + PR c++/91505 + * g++.target/i386/crc32-4.C: New test. + 2019-08-21 Richard Biener <rguenther@suse.de> PR tree-optimization/91482 diff --git a/gcc/testsuite/g++.target/i386/crc32-4.C b/gcc/testsuite/g++.target/i386/crc32-4.C new file mode 100644 index 0000000..542ded6 --- /dev/null +++ b/gcc/testsuite/g++.target/i386/crc32-4.C @@ -0,0 +1,4 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -mavx2" } */ + +#include "../../gcc.target/i386/crc32-4.c" |