IPA: drop implicit_section again
authorMartin Liska <mliska@suse.cz>
Mon, 30 Nov 2020 12:07:27 +0000 (13:07 +0100)
committerMartin Liska <mliska@suse.cz>
Tue, 1 Dec 2020 10:25:34 +0000 (11:25 +0100)
As mentioned in the PR, since 4656461585bfd0b9 implicit_section
was not set to false when set_section was called with the argument
equal to NULL.

gcc/ChangeLog:

PR ipa/98057
* symtab.c (symtab_node::set_section_for_node): Drop
implicit_section if x_section is NULL.

gcc/testsuite/ChangeLog:

PR ipa/98057
* g++.dg/ipa/pr98057.C: New test.

gcc/symtab.c
gcc/testsuite/g++.dg/ipa/pr98057.C [new file with mode: 0644]

index fd7d553..6ceec55 100644 (file)
@@ -1683,7 +1683,10 @@ symtab_node::set_section_for_node (const symtab_node &other)
   if (other.x_section)
     x_section = retain_section_hash_entry (other.x_section);
   else
-    x_section = NULL;
+    {
+      x_section = NULL;
+      implicit_section = false;
+    }
 }
 
 /* Workers for set_section.  */
diff --git a/gcc/testsuite/g++.dg/ipa/pr98057.C b/gcc/testsuite/g++.dg/ipa/pr98057.C
new file mode 100644 (file)
index 0000000..9de9254
--- /dev/null
@@ -0,0 +1,18 @@
+/* PR ipa/98057 */
+/* { dg-do compile } */                                                                        
+/* { dg-options "-O3 -ffunction-sections" } */
+
+class JITSymbolResolver {
+  virtual void anchor();
+};
+class MemoryManager {
+  virtual void anchor();
+};
+class MCJITMemoryManager : MemoryManager {
+  void anchor();
+};
+class RTDyldMemoryManager : MCJITMemoryManager, JITSymbolResolver {
+  void anchor();
+};
+void RTDyldMemoryManager::anchor() {}
+void MCJITMemoryManager::anchor() {}