[PATCH] D75271: [Analyzer][NFC] Add AnalyzerOptions parameter to shouldRegisterXXX() functions

Thu Feb 27 12:23:29 PST 2020

Charusso accepted this revision.
Charusso added a comment.

> ! In D75271#1896223 <https://reviews.llvm.org/D75271#1896223>, @Szelethus wrote:
>  Thinking back, I did have a number of failed attempts to make something a bit less ugly, but the sharp divide between the 2 libraries makes is really-really difficult, and I don't recall alternative solutions being that much better. Either the checker interface gets worse, or the checker registration interface gets so messy that it would severely hurt further improvements in terms of checker dependency development.

Well, if you cannot solve this problem, most likely we neither, but keep in mind:

> If you can't solve a problem, then there is an easier problem you can't solve: find it.

- George Polya

---

As far as you guys keep updating your API I will buy it, thanks!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75271