ipa-cp.c (determine_versionability): Do not create constprop clones, when target_clon...

2016-07-22  Evgeny Stupachenko  <evstupac@gmail.com>

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

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 4f67430ec6f20d3649722432c6046a922967626b..8eb011ccbf358b4386a8d638027dce3eae2330d9 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2016-07-22  Evgeny Stupachenko  <evstupac@gmail.com>
+
+       * ipa-cp.c (determine_versionability): Do not create constprop clones,
+       when target_clones attribute is set.
+
 2016-07-22  Bin Cheng  <bin.cheng@arm.com>
 
        * common.opt (funsafe-loop-optimizations): Mark ignore.

diff --git a/gcc/ipa-cp.c b/gcc/ipa-cp.c
index 4b7f6bbf31c66229372b0445280dbcc5d44e0703..5b6cb9a143f711e5fde0e133a94d6651a2de1d6d 100644 (file)
--- 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 7cef33787ad0fc432513d8f13f15fc725f24e53e..0a48a17c71461d7b9326d517f2e65334c4ab4a64 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2016-07-22  Evgeny Stupachenko  <evstupac@gmail.com>
+
+       * gcc.target/i386/mvc8.c: New test.
+
 2016-07-22  Bin Cheng  <bin.cheng@arm.com>
 
        * 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 (file)
index 0000000..97d7941
--- /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);
+}