diff options
| author | Paula Toth <paulatoth@google.com> | 2020-05-21 01:21:07 -0700 |
|---|---|---|
| committer | Paula Toth <paulatoth@google.com> | 2020-05-21 01:21:37 -0700 |
| commit | b2a485e37edf6415701c472d93afa7fa26dbcccb (patch) | |
| tree | 321dfada318b997dcbede602394517cd67830957 /llvm/lib/Support/StringRef.cpp | |
| parent | dedcefe09d18f1e9d60336541fb6ad03dae551ba (diff) | |
| download | llvm-b2a485e37edf6415701c472d93afa7fa26dbcccb.zip llvm-b2a485e37edf6415701c472d93afa7fa26dbcccb.tar.gz llvm-b2a485e37edf6415701c472d93afa7fa26dbcccb.tar.bz2 | |
[libc] Fix accidental inclusion of system libc headers.
Summary:
I found that because `--system-headers` flag was not included when running clang-tidy, errors produced from compiler provided headers were being suppressed. After passing this flag I realized that by including headers like stdint.h we were indirectly including headers from the system libc. To prevent this we pass `-ffreestanding`.
We don't want to pass `--system-headers` for all checks just the `llvmlibc-restrict-system-libc-headers` therefore we do a separate invocation of clang-tidy for this check.
Reviewers: abrachet, sivachandra
Reviewed By: sivachandra
Subscribers: mgorny, aheejin, tschuett, ecnelises, libc-commits
Tags: #libc-project
Differential Revision: https://reviews.llvm.org/D80178
Diffstat (limited to 'llvm/lib/Support/StringRef.cpp')
0 files changed, 0 insertions, 0 deletions