[PATCH] D54401: [analyzer] Prefer returns values to out-params in CheckerRegistry.cpp

Umann Kristóf via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 12 11:29:54 PST 2018


Szelethus added a comment.

In https://reviews.llvm.org/D54401#1295832, @george.karpenkov wrote:

> > Also, remove diags::note_suggest_disabling_all_checkers.
>
> Isn't that a separate revision? Given that removing existing options is often questionable, I'd much rather see this patch separated.


This isn't an option, just a note :)


Repository:
  rC Clang

https://reviews.llvm.org/D54401





More information about the cfe-commits mailing list