[PATCH] D44064: [Driver] Automatically disable incompatible default sanitizers

2018-03-06 Thread Petr Hosek via Phabricator via cfe-commits
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:

[PATCH] D44064: [Driver] Automatically disable incompatible default sanitizers

2018-03-06 Thread Petr Hosek via Phabricator via cfe-commits
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=137301#toc Repository:

[PATCH] D44064: [Driver] Automatically disable incompatible default sanitizers

2018-03-05 Thread Petr Hosek via Phabricator via cfe-commits
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

[PATCH] D44064: [Driver] Automatically disable incompatible default sanitizers

2018-03-05 Thread Vitaly Buka via Phabricator via cfe-commits
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

[PATCH] D44064: [Driver] Automatically disable incompatible default sanitizers

2018-03-02 Thread Petr Hosek via Phabricator via cfe-commits
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