diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2012-01-26 02:02:57 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2012-01-26 02:02:57 +0000 |
commit | 426fc94ed3bce15b55c43692537e3833388f0352 (patch) | |
tree | 1b0293e3daf2ecea9f68a2e2348d7edf792ba738 /test/CodeGenCXX/pragma-visibility.cpp | |
parent | 027cb30af828f07750f9185782822297a5c57231 (diff) |
Fix our handling of #pragma GCC visibility.
Now the lexer just produces a token and the parser is the one responsible for
activating it.
This fixes problem like the one pr11797 where the lexer and the parser were not
in sync. This also let us be more strict on where in the file we accept
these pragmas.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@149014 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CodeGenCXX/pragma-visibility.cpp')
-rw-r--r-- | test/CodeGenCXX/pragma-visibility.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/test/CodeGenCXX/pragma-visibility.cpp b/test/CodeGenCXX/pragma-visibility.cpp index 2dc8bcc74f..97f8cc8bb8 100644 --- a/test/CodeGenCXX/pragma-visibility.cpp +++ b/test/CodeGenCXX/pragma-visibility.cpp @@ -55,8 +55,6 @@ namespace n __attribute((visibility("default"))) { #pragma GCC visibility pop namespace n __attribute((visibility("default"))) { - extern int foofoo; // FIXME: Shouldn't be necessary, but otherwise the pragma - // gets to Sema before the namespace! #pragma GCC visibility push(hidden) void g() {} // CHECK: define hidden void @_ZN1n1gEv @@ -66,8 +64,6 @@ namespace n __attribute((visibility("default"))) { // We used to test this, but it's insane, so unless it happens in // headers, we should not support it. namespace n __attribute((visibility("hidden"))) { - extern int foofoo; // FIXME: Shouldn't be necessary, but otherwise the pragma - // gets to Sema before the namespace! #pragma GCC visibility pop void h() {} // CHECK disabled: define void @_ZN1n1hEv |