]> granicus.if.org Git - clang/commitdiff
[modules] Properly merge visibility of class definitions that got merged while
authorRichard Smith <richard-llvm@metafoo.co.uk>
Thu, 25 Jun 2015 21:42:33 +0000 (21:42 +0000)
committerRichard Smith <richard-llvm@metafoo.co.uk>
Thu, 25 Jun 2015 21:42:33 +0000 (21:42 +0000)
parsing then merged again when the module was loaded.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@240700 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Serialization/ASTReaderDecl.cpp
test/Modules/Inputs/merge-class-definition-visibility/b.h
test/Modules/Inputs/merge-class-definition-visibility/d.h
test/Modules/Inputs/merge-class-definition-visibility/e.h [new file with mode: 0644]
test/Modules/Inputs/merge-class-definition-visibility/modmap
test/Modules/merge-class-definition-visibility.cpp

index 00ebd3ebe8e441fc4d9e9305bf257d5a3b797cee..f27426bd34bbcd56d93298de05ad047e2abe17ce 100644 (file)
@@ -3897,16 +3897,18 @@ void ASTDeclReader::UpdateDecl(Decl *D, ModuleFile &ModuleFile,
       if (Reader.getContext().getLangOpts().ModulesLocalVisibility) {
         // FIXME: This doesn't send the right notifications if there are
         // ASTMutationListeners other than an ASTWriter.
-        Reader.getContext().mergeDefinitionIntoModule(cast<NamedDecl>(D), Owner,
-                                                      /*NotifyListeners*/false);
-        Reader.PendingMergedDefinitionsToDeduplicate.insert(cast<NamedDecl>(D));
+        Reader.getContext().mergeDefinitionIntoModule(
+            cast<NamedDecl>(Exported), Owner,
+            /*NotifyListeners*/ false);
+        Reader.PendingMergedDefinitionsToDeduplicate.insert(
+            cast<NamedDecl>(Exported));
       } else if (Owner && Owner->NameVisibility != Module::AllVisible) {
         // If Owner is made visible at some later point, make this declaration
         // visible too.
-        Reader.HiddenNamesMap[Owner].push_back(D);
+        Reader.HiddenNamesMap[Owner].push_back(Exported);
       } else {
         // The declaration is now visible.
-        D->Hidden = false;
+        Exported->Hidden = false;
       }
       break;
     }
index 2b8f5f868ed927c92de5f8057cf733b59d6964dc..03c0ad98dfd8d1d37731a18b7b67ded5b5c182c9 100644 (file)
@@ -1,2 +1,4 @@
 // Include definition of A into the same module as c.h
 #include "a.h"
+
+struct B {};
index 2243de1baf9a08cb03712740a09c1e6bdeec00e4..c51edab1416d85fe108427825e64784e70949e89 100644 (file)
@@ -1 +1 @@
-#include "a.h"
+struct B {};
diff --git a/test/Modules/Inputs/merge-class-definition-visibility/e.h b/test/Modules/Inputs/merge-class-definition-visibility/e.h
new file mode 100644 (file)
index 0000000..f126b50
--- /dev/null
@@ -0,0 +1,3 @@
+#include "a.h"
+
+struct B {};
index 7d988fbba0039971eafa6ad6a8ded37b4c26fff9..dcb65871886a71051989b6a89ef766e141ab0d4e 100644 (file)
@@ -3,5 +3,6 @@ module Def1 {
   module C { header "c.h" }
 }
 module Def2 {
-  header "d.h"
+  module D { header "d.h" }
+  module E { header "e.h" }
 }
index e8602c0d5489d2ece2ae59c926700cfd6915e7bb..ac4c951584498fd7d6c3b3f3a109c7e0ce426de4 100644 (file)
@@ -1,15 +1,23 @@
 // RUN: rm -rf %t
 // RUN: %clang_cc1 -fmodules -fmodule-map-file=%S/Inputs/merge-class-definition-visibility/modmap \
 // RUN:            -I%S/Inputs/merge-class-definition-visibility \
-// RUN:            -fmodules-cache-path=%t %s -verify
+// RUN:            -fmodules-cache-path=%t %s -verify \
+// RUN:            -fmodules-local-submodule-visibility
 // expected-no-diagnostics
 
 #include "c.h"
 template<typename T> struct X { T t; };
 typedef X<A> XA;
+struct B;
 
-#include "d.h"
-// Ensure that this triggers the import of the second definition from d.h,
+#include "e.h"
+// Ensure that this triggers the import of the second definition from e.h,
 // which is necessary to make the definition of A visible in the template
 // instantiation.
 XA xa;
+
+// Ensure that we make the definition of B visible. We made the parse-merged
+// definition from e.h visible, which makes the definition from d.h visible,
+// and that definition was merged into the canonical definition from b.h,
+// so that becomes visible, and we have a visible definition.
+B b;