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

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 2 19:53:13 PDT 2019


phosek updated this revision to Diff 213170.
phosek edited the summary of this revision.
phosek added a comment.
Herald added a subscriber: cryptoad.

I've updated the change and renamed the flag to `-fsanitize-no-runtime` although I'm not sure if it's necessarily better. I was thinking that ideal interface might be `-fsanitize-runtime=(none|minimal|shared|static)` which could also replace `-fshared-libsan` and `-fstatic-libsan`, but it'd be a breaking change so we might have to keep the existing flag and maybe print a deprecation warning. WDYT?


Repository:
  rC Clang

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

https://reviews.llvm.org/D65029

Files:
  clang/include/clang/Driver/Options.td
  clang/include/clang/Driver/SanitizerArgs.h
  clang/lib/Driver/SanitizerArgs.cpp
  clang/test/Driver/sanitizer-ld.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65029.213170.patch
Type: text/x-patch
Size: 8355 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190803/5d63a57d/attachment.bin>


More information about the cfe-commits mailing list