diff options
author | Richard Smith <richard-llvm@metafoo.co.uk> | 2015-03-17 02:23:11 +0000 |
---|---|---|
committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2015-03-17 02:23:11 +0000 |
commit | e146d8df5bebd293af10c6513319106f0b3713f1 (patch) | |
tree | 38a10f21cff1df4f70b8fde7186feada9cc4f768 /test/Modules/anon-namespace.cpp | |
parent | 55d9d4a934eacffbcfbd3073b71aeacf6c576d38 (diff) |
[modules] Fix bug where an anonymous namespace could cause the containing
namespace to not merge properly.
We have an invariant here: after a declaration reads its canonical declaration,
it can assume the canonical declaration is fully merged. This invariant can be
violated if deserializing some declaration triggers the deserialization of a
later declaration, because that later declaration can in turn deserialize a
redeclaration of that first declaration before it is fully merged.
The anonymous namespace for a namespace gets stored with the first declaration
of that namespace, which may be before its parent namespace, so defer loading
it until after we've finished merging the surrounding namespace.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@232455 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/Modules/anon-namespace.cpp')
-rw-r--r-- | test/Modules/anon-namespace.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/test/Modules/anon-namespace.cpp b/test/Modules/anon-namespace.cpp new file mode 100644 index 0000000000..6c085ebc88 --- /dev/null +++ b/test/Modules/anon-namespace.cpp @@ -0,0 +1,6 @@ +// RUN: rm -rf %t +// RUN: %clang_cc1 -fmodules -fmodules-cache-path=%t -I%S/Inputs/anon-namespace -verify %s +// expected-no-diagnostics +#include "b1.h" +#include "c.h" +using namespace N; |