ipa-cp.c (determine_versionability): Do not create constprop clones, when target_clones attribute is set.

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

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.

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 ())

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.

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);
+}