diff options
author | Alex Lorenz <arphaman@gmail.com> | 2017-07-26 12:20:57 +0000 |
---|---|---|
committer | Alex Lorenz <arphaman@gmail.com> | 2017-07-26 12:20:57 +0000 |
commit | 5d48424a30960cce0b5b178a284eedab1c16c79c (patch) | |
tree | a25f4d2e0e3f8d81e10a6fd8a202522684667589 /clang/test/Parser/pragma-options.cpp | |
parent | 0b7bf7a2e3cb34086d6a05419319fd35ae8dd9a8 (diff) | |
download | bcm5719-llvm-5d48424a30960cce0b5b178a284eedab1c16c79c.tar.gz bcm5719-llvm-5d48424a30960cce0b5b178a284eedab1c16c79c.zip |
Recommit r308327 2nd time: Add a warning for missing
'#pragma pack (pop)' and suspicious uses of '#pragma pack' in included files
The first recommit (r308441) caused a "non-default #pragma pack value might
change the alignment of struct or union members in the included file" warning
in LLVM itself. This recommit tweaks the added warning to avoid warnings for
#includes that don't have any records that are affected by the non-default
alignment. This tweak avoids the previously emitted warning in LLVM.
Original message:
This commit adds a new -Wpragma-pack warning. It warns in the following cases:
- When a translation unit is missing terminating #pragma pack (pop) directives.
- When entering an included file if the current alignment value as determined
by '#pragma pack' directives is different from the default alignment value.
- When leaving an included file that changed the state of the current alignment
value.
rdar://10184173
Differential Revision: https://reviews.llvm.org/D35484
llvm-svn: 309106
Diffstat (limited to 'clang/test/Parser/pragma-options.cpp')
-rw-r--r-- | clang/test/Parser/pragma-options.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/test/Parser/pragma-options.cpp b/clang/test/Parser/pragma-options.cpp index 84cd38dfb3c..8f5a2152c7b 100644 --- a/clang/test/Parser/pragma-options.cpp +++ b/clang/test/Parser/pragma-options.cpp @@ -1,4 +1,4 @@ -// RUN: %clang_cc1 -triple i386-apple-darwin9 -fsyntax-only -verify %s +// RUN: %clang_cc1 -triple i386-apple-darwin9 -Wno-pragma-pack -fsyntax-only -verify %s // expected-no-diagnostics class C { |