From 58b3986e3458ed69301e4128fd704598ad01cb89 Mon Sep 17 00:00:00 2001 From: Evgeny Stupachenko Date: Fri, 22 Jul 2016 15:00:42 +0000 Subject: [PATCH] ipa-cp.c (determine_versionability): Do not create constprop clones, when target_clones attribute is set. 2016-07-22 Evgeny Stupachenko gcc/ * ipa-cp.c (determine_versionability): Do not create constprop clones, when target_clones attribute is set. gcc/testsuite/ * gcc.target/i386/mvc8.c: New test. From-SVN: r238645 --- gcc/ChangeLog | 5 +++++ gcc/ipa-cp.c | 7 +++++++ gcc/testsuite/ChangeLog | 4 ++++ gcc/testsuite/gcc.target/i386/mvc8.c | 14 ++++++++++++++ 4 files changed, 30 insertions(+) create mode 100644 gcc/testsuite/gcc.target/i386/mvc8.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 4f67430ec6f..8eb011ccbf3 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2016-07-22 Evgeny Stupachenko + + * ipa-cp.c (determine_versionability): Do not create constprop clones, + when target_clones attribute is set. + 2016-07-22 Bin Cheng * common.opt (funsafe-loop-optimizations): Mark ignore. diff --git a/gcc/ipa-cp.c b/gcc/ipa-cp.c index 4b7f6bbf31c..5b6cb9a143f 100644 --- a/gcc/ipa-cp.c +++ b/gcc/ipa-cp.c @@ -533,6 +533,13 @@ determine_versionability (struct cgraph_node *node, coexist, but that may not be worth the effort. */ reason = "function has SIMD clones"; } + else if (lookup_attribute ("target_clones", DECL_ATTRIBUTES (node->decl))) + { + /* Ideally we should clone the target clones themselves and create + copies of them, so IPA-cp and target clones can happily + coexist, but that may not be worth the effort. */ + reason = "function target_clones attribute"; + } /* Don't clone decls local to a comdat group; it breaks and for C++ decloned constructors, inlining is always better anyway. */ else if (node->comdat_local_p ()) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 7cef33787ad..0a48a17c714 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2016-07-22 Evgeny Stupachenko + + * gcc.target/i386/mvc8.c: New test. + 2016-07-22 Bin Cheng * gcc.dg/tree-ssa/pr19210-1.c: Refine test strings. diff --git a/gcc/testsuite/gcc.target/i386/mvc8.c b/gcc/testsuite/gcc.target/i386/mvc8.c new file mode 100644 index 00000000000..97d7941a5ab --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/mvc8.c @@ -0,0 +1,14 @@ +/* { dg-do compile } */ +/* { dg-require-ifunc "" } */ +/* { dg-options "-O3 -fno-inline" } */ +/* { dg-final { scan-assembler-not "constprop" } } */ +__attribute__((target_clones("arch=core-avx2","arch=slm","default"))) +void foo (float *a, int b) { + *a = (float)b; +} +float a; +int main() { + int i; + for (i = 0; i < 1024; i++) + foo (&a, 5); +} -- 2.30.2