[PATCH] D65029: [Driver] Support for disabling sanitizer runtime linking

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.

Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65029/new/

https://reviews.llvm.org/D65029