[PATCH] D53995: [analyzer] Drastically simplify the tblgen files used for checkers

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 9 14:59:35 PST 2018


NoQ accepted this revision.
NoQ added a comment.

Thx!! Burn it.

> tblgen files look awesome. I get that this isn't a very strong point, but it's a pretty sight compared to a .def file.

I agree that it's aesthetically satisfying, but it is (1) inconvenient to write because that's the whole new syntax you need to memorize, i.e. all those <> and semicolons and (2) not cooperating when i want to look up checker name by source file (have to scan a few pages up to see the alias for the package and then jump multiple times to see what it expands to). So i'm still for ditching tblgen eventually :) But i definitely don't insist.


Repository:
  rC Clang

https://reviews.llvm.org/D53995





More information about the cfe-commits mailing list