[PATCH] D65029: [Driver] Support for disabling sanitizer runtime linking
Petr Hosek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Jul 20 18:36:21 PDT 2019
phosek added a comment.
In D65029#1594437 <https://reviews.llvm.org/D65029#1594437>, @pcc wrote:
> It's also worth asking whether `-nodefaultlibs` would work for this use case as @filcab suggested on D64547 <https://reviews.llvm.org/D64547>.
It wouldn't, `-nodefaultlibs` (same as `-nostdlib`) disables linking of runtime dependencies, but not the runtime itself, e.g. `-fsanitize=address -nodefaultlibs` disables linking of `-lpthread -lrt -lm -ldl` which are all dependencies of ASan, but doesn't disable linking of `libclang_rt.asan.a` which is what we care about.
In D65029#1594409 <https://reviews.llvm.org/D65029#1594409>, @MaskRay wrote:
> There is an existing option `-fsanitize-minimal-runtime`, will `-fsanitize-no-runtime` be a name providing more consistency?
That's fine with me, I used `-f[no-]sanitize-link-deps` for consistency with XRay, but `-fsanitize-no-runtime` would work as well.
CHANGES SINCE LAST ACTION
More information about the cfe-commits