[PATCH] disable IPA-cp cloning for functions with target_clones attribute

[Evgeny Stupachenko]

Hi,

Fix ICE when IPA-cp and target_clones are applied to the same function.
Is the patch ok for trunk?

Thanks,
Evgeny

2016-06-24  Evgeny Stupachenko  <evstu...@gmail.com>

gcc/
        * ipa-cp.c (determine_versionability): Do not create constprop clones,
        when target_clones attribute is set.
diff --git a/gcc/ipa-cp.c b/gcc/ipa-cp.c
index 2710494..4b642ba 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/gcc.target/i386/mvc8.c
b/gcc/testsuite/gcc.target/i386/mvc8.c
new file mode 100644
index 0000000..e9ab9e1
--- /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);
+}