[PATCH] D46159: [clang-tidy] Add a flag to enable alpha checkers

Paul Fultz II via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 3 07:59:06 PDT 2018


pfultz2 added inline comments.


================
Comment at: clang-tidy/ClangTidy.cpp:373-376
   // FIXME: Remove this option once clang's cfg-temporary-dtors option defaults
   // to true.
   AnalyzerOptions->Config["cfg-temporary-dtors"] =
       Context.getOptions().AnalyzeTemporaryDtors ? "true" : "false";
----------------
alexfh wrote:
> alexfh wrote:
> > NoQ wrote:
> > > Btw this is already enabled by default.
> > Should we kill the flag completely?
> I've removed the clang-tidy configuration option in r331456.
I didnt modify this line of code. Are you just wanting me to rebase?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D46159





More information about the cfe-commits mailing list