diff options
| author | Anna Zaks <ganna@apple.com> | 2016-09-15 21:02:18 +0000 |
|---|---|---|
| committer | Anna Zaks <ganna@apple.com> | 2016-09-15 21:02:18 +0000 |
| commit | 691644f3ca6e7b0cd3acc0cf8ddde0e200bec2ae (patch) | |
| tree | 1a0866389a84d9c65ebb5fb91501650911e317ec /clang/lib/Frontend/InitPreprocessor.cpp | |
| parent | 592bc4553306b250efe82ad37a0824addc1bf4ba (diff) | |
| download | llvm-691644f3ca6e7b0cd3acc0cf8ddde0e200bec2ae.zip llvm-691644f3ca6e7b0cd3acc0cf8ddde0e200bec2ae.tar.gz llvm-691644f3ca6e7b0cd3acc0cf8ddde0e200bec2ae.tar.bz2 | |
[compiler-rt] Do not introduce __sanitizer namespace globally
The definitions in sanitizer_common may conflict with definitions from system headers because:
The runtime includes the system headers after the project headers (as per LLVM coding guidelines).
lib/sanitizer_common/sanitizer_internal_defs.h pollutes the namespace of everything defined after it, which is all/most of the sanitizer .h and .cc files and the included system headers with: using namespace __sanitizer; // NOLINT
This patch solves the problem by introducing the namespace only within the sanitizer namespaces as proposed by Dmitry.
Differential Revision: https://reviews.llvm.org/D21947
llvm-svn: 281657
Diffstat (limited to 'clang/lib/Frontend/InitPreprocessor.cpp')
0 files changed, 0 insertions, 0 deletions