[PATCH] D52524: Add -Wpoison-system-directories warning
Sam Clegg via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 27 15:32:43 PDT 2021
sbc100 added a comment.
Would it make sense to have this apply to system library paths too? i.e. `-L/usr/local/lib`.
We have a bespoke mechanism in emscripten right now that is a bit more flexible but I'd love to simple replace it with `-Wpoison-system-directories`.
The name "system directories" seems to imply that it should apply to both include and library directories. It also looks like the gcc version applies to both `Warn for -I and -L options using system directories if cross compiling` (from https://github.com/Metrological/buildroot/blob/master/package/gcc/4.8.4/910-gcc-poison-system-directories.patch).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D52524/new/
https://reviews.llvm.org/D52524
More information about the cfe-commits
mailing list