[PATCH] D102149: [analyzer][ctu] Allow loading invocation list from a compilation database automatically detected from the ctu-dir

Ella Ma via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 18 03:08:49 PDT 2021


OikawaKirie added a comment.

In D102149#2765402 <https://reviews.llvm.org/D102149#2765402>, @martong wrote:

> Thank you @OikawaKirie for working on this many CTU related patches! I am going to find time for a thorough review and going to pursue @gamesh411 as well to do the same! On the other hand, it would be really useful if you could build a "Stack" from these patches, I mean could you please set which patch depends from which other (see "Edit related revisions")?

To make it easier to review, all these recently submitted patches are based on the `main` branch. Their code does not depend on each other.

Logically, they can be separated into two branches: invocation list and ctu index.
The invocation list related patches target on constructing the invocation list in ASTLoader from a compilation database. And the problems of using invocation lists are also fixed.
Whereas the ctu index related patches mainly focus on the delimiter character in the ctu index, which is generated by the `clang-extdef-mapping` tool.

If the "stack" means the logical relations, I will update that later. Otherwise, it is OK to start with any of the patches.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102149



More information about the cfe-commits mailing list