diff options
author | Richard Smith <richard-llvm@metafoo.co.uk> | 2019-04-18 00:56:58 +0000 |
---|---|---|
committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2019-04-18 00:56:58 +0000 |
commit | c2e330a40a66179aa8ee3172b96e0c40dd31593e (patch) | |
tree | 06ffa5cf0dafebe84d4799c8271c80b47a0cba50 /test/CXX | |
parent | e9149d19acaa08b5de55ecbd19348cbb60cdde72 (diff) |
[c++2a] Improve diagnostic for use of declaration from another TU's
global module fragment.
We know that the declaration in question should have been introduced by
a '#include', so try to figure out which one and suggest it. Don't
suggest importing the global module fragment itself!
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@358631 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CXX')
-rw-r--r-- | test/CXX/basic/basic.scope/basic.scope.namespace/p2.cpp | 86 |
1 files changed, 86 insertions, 0 deletions
diff --git a/test/CXX/basic/basic.scope/basic.scope.namespace/p2.cpp b/test/CXX/basic/basic.scope/basic.scope.namespace/p2.cpp new file mode 100644 index 0000000000..55c5ce6c4c --- /dev/null +++ b/test/CXX/basic/basic.scope/basic.scope.namespace/p2.cpp @@ -0,0 +1,86 @@ +// RUN: rm -rf %t +// RUN: mkdir -p %t +// RUN: echo '#ifndef FOO_H' > %t/foo.h +// RUN: echo '#define FOO_H' >> %t/foo.h +// RUN: echo 'extern int in_header;' >> %t/foo.h +// RUN: echo '#endif' >> %t/foo.h +// RUN: %clang_cc1 -std=c++2a -I%t -emit-module-interface -DINTERFACE %s -o %t.pcm +// RUN: %clang_cc1 -std=c++2a -I%t -fmodule-file=%t.pcm -DIMPLEMENTATION %s -verify -fno-modules-error-recovery +// RUN: %clang_cc1 -std=c++2a -I%t -fmodule-file=%t.pcm %s -verify -fno-modules-error-recovery + +#ifdef INTERFACE +module; +#include "foo.h" +int global_module_fragment; +export module A; +export int exported; +int not_exported; +static int internal; + +module :private; +int not_exported_private; +static int internal_private; +#else + +#ifdef IMPLEMENTATION +module; +#endif + +void test_early() { + in_header = 1; // expected-error {{missing '#include "foo.h"'; 'in_header' must be declared before it is used}} + // expected-note@*{{previous}} + + global_module_fragment = 1; // expected-error {{missing '#include'; 'global_module_fragment' must be declared before it is used}} + + exported = 1; // expected-error {{must be imported from module 'A'}} + // expected-note@p2.cpp:16 {{previous}} + + not_exported = 1; // expected-error {{undeclared identifier}} + + internal = 1; // expected-error {{undeclared identifier}} + + not_exported_private = 1; // expected-error {{undeclared identifier}} + + internal_private = 1; // expected-error {{undeclared identifier}} +} + +#ifdef IMPLEMENTATION +module A; +#else +import A; +#endif + +void test_late() { + in_header = 1; // expected-error {{missing '#include "foo.h"'; 'in_header' must be declared before it is used}} + // expected-note@*{{previous}} + + global_module_fragment = 1; // expected-error {{missing '#include'; 'global_module_fragment' must be declared before it is used}} + + exported = 1; + + not_exported = 1; +#ifndef IMPLEMENTATION + // expected-error@-2 {{undeclared identifier 'not_exported'; did you mean 'exported'}} + // expected-note@p2.cpp:16 {{declared here}} +#endif + + internal = 1; +#ifndef IMPLEMENTATION + // FIXME: should not be visible here + // expected-error@-3 {{undeclared identifier}} +#endif + + not_exported_private = 1; +#ifndef IMPLEMENTATION + // FIXME: should not be visible here + // expected-error@-3 {{undeclared identifier}} +#endif + + internal_private = 1; +#ifndef IMPLEMENTATION + // FIXME: should not be visible here + // expected-error@-3 {{undeclared identifier}} +#endif +} + +#endif |