[PATCH] D71612: [analyzer] Add PlacementNewChecker
Gabor Marton via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 10 09:27:37 PST 2020
martong added a comment.
In D71612#1814225 <https://reviews.llvm.org/D71612#1814225>, @NoQ wrote:
> In D71612#1813937 <https://reviews.llvm.org/D71612#1813937>, @martong wrote:
>
> > I am going to execute the analysis on LLVM/Clang as you suggested, so then we'll see what the experiment will bring us. And then we could enable it by default I think. BTW how can we do that? I could not find any default enablement related config in Checkers.td ...
>
>
> Enabling checkers by default is controlled by the messy code in `RenderAnalyzerOptions()` in the Driver.
>
> Wait, you already made it on-by-default. Checkers that are currently under development go into the `alpha` package.
Ups, I am sorry. Now I am creating another commit which moves it to alpha. (So then, If I understand it correctly then the `cplusplus` is enabled by default, but `alpha` is not.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71612/new/
https://reviews.llvm.org/D71612
More information about the cfe-commits
mailing list