[PATCH] D37383: [AST] Add TableGen for StmtDataCollectors

Raphael Isemann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 1 10:32:37 PDT 2017

teemperor added a comment.

@arphaman I suggested tablegen in https://reviews.llvm.org/D36664 because I remembered we had some CMake sanity check about not having an *.inc in our include dir: https://github.com/llvm-mirror/clang/blob/master/CMakeLists.txt#L266 Not sure if it actually fires for our case, but it looks like it does.

Also it would be nicer to maintain once we have a second attribute for an optional single-TU code as tablegen could just fall back from `SingleTUCode` to the normal `Code` attribute. And we would have an easy solution for the CrossTU/SingleTU inc files as tablegen could just create both without us having to manually copy code around or do preprocessor hacks.

I guess this review will make more sense once we have both CrossTU and SingleTU code ready.


More information about the cfe-commits mailing list