[C++] compiler incompatibility with lambdas

https://gcc.gnu.org/ml/gcc-patches/2019-03/msg00982.html
	* lambda.c (maybe_add_lambda_conv_op): Don't add to comdat group.
	* g++.dg/abi/lambda-static-1.C: New.

From-SVN: r269915

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 16f1b8e1aaedf0dfd8aac7fc8d829ae1c0f91bce..7589b30e8522446a888d510586f1c5cc718fcdf6 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,7 @@
+2019-03-25  Nathan Sidwell  <nathan@acm.org>
+
+       * lambda.c (maybe_add_lambda_conv_op): Don't add to comdat group.
+
 2019-03-22  Jakub Jelinek  <jakub@redhat.com>
 
        PR c++/60702

diff --git a/gcc/cp/lambda.c b/gcc/cp/lambda.c
index 3e176760ba5c31d3d88fb5416fccab4f240e5e81..93664181b47b949d6cc5f66e198b091b5c3d499d 100644 (file)
--- a/gcc/cp/lambda.c
+++ b/gcc/cp/lambda.c
@@ -1252,12 +1252,6 @@ maybe_add_lambda_conv_op (tree type)
 
   start_preparsed_function (statfn, NULL_TREE,
                            SF_PRE_PARSED | SF_INCLASS_INLINE);
-  if (DECL_ONE_ONLY (statfn))
-    {
-      /* Put the thunk in the same comdat group as the call op.  */
-      cgraph_node::get_create (statfn)->add_to_same_comdat_group
-       (cgraph_node::get_create (callop));
-    }
   tree body = begin_function_body ();
   tree compound_stmt = begin_compound_stmt (0);
   if (!generic_lambda_p)

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index ef7ae1b1c4c6e51d39a448f1a9a5c2579894460a..5aead43f1475ab53c8a84be1ecd11ece22d108c2 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2019-03-25  Nathan Sidwell  <nathan@acm.org>
+
+       * g++.dg/abi/lambda-static-1.C: New.
+
 2019-03-25  Richard Biener  <rguenther@suse.de>
 
        PR tree-optimization/89802

diff --git a/gcc/testsuite/g++.dg/abi/lambda-static-1.C b/gcc/testsuite/g++.dg/abi/lambda-static-1.C
new file mode 100644 (file)
index 0000000..3d23ddc
--- /dev/null
+++ b/gcc/testsuite/g++.dg/abi/lambda-static-1.C
@@ -0,0 +1,25 @@
+// { dg-do compile { target { c++14 && comdat_group } } }
+// { dg-additional-options -fno-inline }
+
+inline auto lamby () 
+{
+  return [] {};
+}
+
+void direct ()
+{
+  lamby ()();
+}
+
+void indirect ()
+{
+  void (*invoke) () = lamby ();
+
+  invoke ();
+}
+
+// The call operator and the static invoker should be comdat, but not
+// the same group.  (that would be a compiler incompatibility)
+
+// { dg-final { scan-assembler ".section\[\t ]*.text._ZZ5lambyvENKUlvE_clEv,\[^\n\r]*,_ZZ5lambyvENKUlvE_clEv,comdat" } }
+// { dg-final { scan-assembler ".section\[\t ]*.text._ZZ5lambyvENUlvE_4_FUNEv,\[^\n\r]*,_ZZ5lambyvENUlvE_4_FUNEv,comdat" } }