This revision was automatically updated to reflect the committed changes.
Closed by commit rL326860: [Driver] Automatically disable incompatible default
sanitizers (authored by phosek, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D44
This revision was automatically updated to reflect the committed changes.
Closed by commit rC326860: [Driver] Automatically disable incompatible default
sanitizers (authored by phosek, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44064?vs=136893&id=137301#toc
Repository:
phosek added inline comments.
Comment at: clang/lib/Driver/SanitizerArgs.cpp:369
// Check that LTO is enabled if we need it.
if ((Kinds & NeedsLTO) && !D.isUsingLTO()) {
D.Diag(diag::err_drv_argument_only_allowed_with)
vitalybuka wrote:
> Is this chang
vitalybuka added inline comments.
Comment at: clang/lib/Driver/SanitizerArgs.cpp:369
// Check that LTO is enabled if we need it.
if ((Kinds & NeedsLTO) && !D.isUsingLTO()) {
D.Diag(diag::err_drv_argument_only_allowed_with)
Is this change needed to avoi
phosek created this revision.
phosek added reviewers: kcc, vitalybuka, mcgrathr.
Herald added subscribers: cfe-commits, cryptoad.
phosek added a dependent revision: D44065: [Driver] Enable SafeStack by default
on Fuchsia.
When a sanitizer incompatible with one of the default sanitizers
is explici