]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
ipa/106124 - ICE with -fkeep-inline-functions and OpenMP
authorRichard Biener <rguenther@suse.de>
Mon, 27 Mar 2023 14:40:15 +0000 (16:40 +0200)
committerRichard Biener <rguenther@suse.de>
Wed, 29 Mar 2023 06:55:34 +0000 (08:55 +0200)
The testcases in this bug reveal cases where an early generated
type is collected because it was unused but gets attempted to
be recreated later when a late DIE for a function (an OpenMP
reduction) is created.  That's unexpected and possibly the fault
of OpenMP but the following allows the re-creation of the context
type to succeed.

PR ipa/106124
* dwarf2out.cc (lookup_type_die): Reset TREE_ASM_WRITTEN
so we can re-create the DIE for the type if required.

* g++.dg/gomp/pr106124.C: New testcase.

gcc/dwarf2out.cc
gcc/testsuite/g++.dg/gomp/pr106124.C [new file with mode: 0644]

index 1711ad2c2da319e1917bfb256c35802f7a10e894..1a0015ce00ffbdf3f0ebed7044ec23e049d3d035 100644 (file)
@@ -5894,6 +5894,7 @@ lookup_type_die (tree type)
   if (die && die->removed)
     {
       TYPE_SYMTAB_DIE (type) = NULL;
+      TREE_ASM_WRITTEN (type) = 0;
       return NULL;
     }
   return die;
diff --git a/gcc/testsuite/g++.dg/gomp/pr106124.C b/gcc/testsuite/g++.dg/gomp/pr106124.C
new file mode 100644 (file)
index 0000000..3129749
--- /dev/null
@@ -0,0 +1,19 @@
+// { dg-do compile }
+// { dg-require-effective-target c++11 }
+// { dg-options "-g -O2 -fopenmp -fkeep-inline-functions" }
+
+int q;
+struct A
+{
+  typedef int T;
+#pragma omp declare reduction (x : T : omp_out += omp_in + [] (){ return q; }()) initializer (omp_priv = [](){ return 0; }())
+  static void foo ();
+};
+void bar (int &, int &);
+void
+A::foo ()
+{
+  int r = 0, s = 0;
+#pragma omp parallel reduction (x : r, s)
+  bar (r, s);
+}