[PATCH] D60059: [Driver] implement -feverything

2019-04-01 Thread Peter Collingbourne via Phabricator via cfe-commits
pcc added a comment. In D60059#1450320 , @lebedev.ri wrote: > In D60059#1450252 , @yuxuanchen1997 > wrote: > > > > enable all runtime sanitizers for extra safety > > > > Brilliant idea. But I wonder how that could

[PATCH] D60059: [Driver] implement -feverything

2019-04-01 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. In D60059#1450252 , @yuxuanchen1997 wrote: > > enable all runtime sanitizers for extra safety > > Brilliant idea. But I wonder how that could be done for -fsanitize. ASAN and > MSAN can't be used together. Nonsense. There is

[PATCH] D60059: [Driver] implement -feverything

2019-04-01 Thread Yuxuan Chen via Phabricator via cfe-commits
yuxuanchen1997 requested changes to this revision. yuxuanchen1997 added a comment. This revision now requires changes to proceed. > enable all runtime sanitizers for extra safety Brilliant idea. But I wonder how that could be done for -fsanitize. ASAN and MSAN can't be used together. Repositor

[PATCH] D60059: [Driver] implement -feverything

2019-04-01 Thread Stephen Hines via Phabricator via cfe-commits
srhines added a comment. In addition to -fno-everything, don't forget about -fnothing and -fno-nothing, which seem like they would also be nice to have. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60059/new/ https://reviews.llvm.org/D60059 __

[PATCH] D60059: [Driver] implement -feverything

2019-04-01 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers created this revision. nickdesaulniers added a reviewer: rsmith. Herald added subscribers: llvm-commits, cfe-commits, jdoerfert. Herald added projects: clang, LLVM. Due to the wild popularity of -Weverything (used for enabling all warnings), we're back for round 2 (electric boogalo