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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
