Szelethus added a reviewer: aaron.ballman.
Szelethus added a comment.

Aaron, you recently changed these file too, if you have the time, I'll happily 
listen to your feedback :)

Also, I was thinking that maybe we could just `std::move` 
`CheckerRegisrty::Checkers` and `CheckerRegisrty::Packages` to 
`AnalyzerOptions` in `CheckerRegisrty`'s destructor, which will finally enable 
us to access all registered checkers, and a bunch of other things (like whether 
they are enabled, what options they have, etc).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57855/new/

https://reviews.llvm.org/D57855



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to