[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