diff options
| author | Fangrui Song <i@maskray.me> | 2024-01-11 18:30:16 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-01-11 18:30:16 -0800 |
| commit | 6a0c440159aa2b2e895fe6e04af39c91e54733ce (patch) | |
| tree | 172a6d44b3bff33d50a7632f4a2c967540bf2bf4 /llvm/lib/Support/StringRef.cpp | |
| parent | 9095eec0524d39d447d6f94cd3f9896cc5fc656f (diff) | |
| download | llvm-6a0c440159aa2b2e895fe6e04af39c91e54733ce.zip llvm-6a0c440159aa2b2e895fe6e04af39c91e54733ce.tar.gz llvm-6a0c440159aa2b2e895fe6e04af39c91e54733ce.tar.bz2 | |
[Driver,sanitizer] Remove RequiresPIE and msan's NeedPIE setting (#77689)
The two variables cause clang to default to -fPIE when no PIC/PIC option
is
specified.
msan used to require PIE because many `kMemoryLayout` made the low
address (used by ET_EXEC executables) invalid. Current msan.h no longer
does so, rendering this PIE requirement unneeded. The same argument
applies to -fsanitize=dataflow.
On Linux, most builds set CLANG_DEFAULT_PIE_ON_LINUX to 1, making
`RequiresPIE/NeedPIE` redundant on Linux.
(`NeedPIE` is not removed for now due to the -fsanitize-cfi-cross-dso
comment. If it's indeed incompatible with explicit -fno-pic, a warning
is probably better.)
Diffstat (limited to 'llvm/lib/Support/StringRef.cpp')
0 files changed, 0 insertions, 0 deletions