[PATCH] D104439: [analyzer][NFC] Demonstrate a move from the analyzer-configs `.def` file to a TableGen file

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 22 06:19:00 PDT 2021


vsavchenko added a comment.

In D104439#2832984 <https://reviews.llvm.org/D104439#2832984>, @Szelethus wrote:

> Ping.
>
> The main question with this patch really is whether we want TableGen at all or not, as for what we generate precisely and how we'd utilize it is a discussion for later.

I don't really have strong opinions here.  On one hand, I'd prefer TableGen to a purely macro-based solution.  On the other hand, something, let's say, C++-native (like `cl::opt`) seems better than TableGen.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104439



More information about the cfe-commits mailing list