[PATCH] D137197: [clang][modules][deps] Transitive module maps are not affecting

2022-12-01 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGed07fe71d7bb: [clang][modules][deps] Transitive module maps are not affecting (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D13719

[PATCH] D137197: [clang][modules][deps] Transitive module maps are not affecting

2022-11-08 Thread Michael Spencer via Phabricator via cfe-commits
Bigcheese accepted this revision. Bigcheese added a comment. This revision is now accepted and ready to land. Yeah, I agree with this. lgtm. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137197/new/ https://reviews.llvm.org/D137197 ___

[PATCH] D137197: [clang][modules][deps] Transitive module maps are not affecting

2022-11-01 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 updated this revision to Diff 472418. jansvoboda11 added a comment. Extract lambda, replace `while` with `for` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137197/new/ https://reviews.llvm.org/D137197 Files: clang/lib/Serialization

[PATCH] D137197: [clang][modules][deps] Transitive module maps are not affecting

2022-11-01 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, vsapsai. Herald added a subscriber: ributzka. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Currently, the algorithm fo