[PATCH] D34512: [libTooling] Add preliminary Cross Translation Unit support for libTooling

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 22 07:25:15 PDT 2017

klimek added a comment.

General direction looks good.

I think in addition to integration tests through the static analyzer, we should still have unit tests.

Comment at: include/clang/Tooling/CrossTranslationUnit.h:47
+  const FunctionDecl *getCTUDefinition(const FunctionDecl *FD, StringRef CTUDir,
+                                       StringRef IndexName);
I'd just spell out Cross & s/CTUDIR/IndexDir/ perhaps? (if CTUDir is to contain the indices)
I think as this is the major API entry point, a comment that explains the meaning of the arguments and how to use them would be nice.


More information about the cfe-commits mailing list