forked from OSchip/llvm-project
45b4014711
'#pragma pack (pop)' and suspicious uses of '#pragma pack' in included files The second recommit (r309106) was reverted because the "non-default #pragma pack value chages the alignment of struct or union members in the included file" warning proved to be too aggressive for external projects like Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=749197). This recommit makes the problematic warning a non-default one, and gives it the -Wpragma-pack-suspicious-include warning option. 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: 309386 |
||
|---|---|---|
| .. | ||
| ASTCommon.cpp | ||
| ASTCommon.h | ||
| ASTReader.cpp | ||
| ASTReaderDecl.cpp | ||
| ASTReaderInternals.h | ||
| ASTReaderStmt.cpp | ||
| ASTWriter.cpp | ||
| ASTWriterDecl.cpp | ||
| ASTWriterStmt.cpp | ||
| CMakeLists.txt | ||
| GeneratePCH.cpp | ||
| GlobalModuleIndex.cpp | ||
| Module.cpp | ||
| ModuleFileExtension.cpp | ||
| ModuleManager.cpp | ||
| MultiOnDiskHashTable.h | ||