[PATCH] D134222: [clang][deps] Report module map describing compiled module

w2yehia added a comment. In D134222#3815956 , @jansvoboda11 wrote: > In D134222#3815793 , @w2yehia wrote: > >> @jansvoboda11 >> FYI, this is breaking AIX buildbot: >> https://lab.llvm.org/buildbot/#/builders/214

[PATCH] D134222: [clang][deps] Report module map describing compiled module

jansvoboda11 added a comment. In D134222#3815793 , @w2yehia wrote: > @jansvoboda11 > FYI, this is breaking AIX buildbot: > https://lab.llvm.org/buildbot/#/builders/214/builds/3436/steps/6/logs/FAIL__Clang__modules-header-sharing_m Thanks for reporting t

[PATCH] D134222: [clang][deps] Report module map describing compiled module

w2yehia added a comment. @jansvoboda11 FYI, this is breaking AIX buildbot: https://lab.llvm.org/buildbot/#/builders/214/builds/3436/steps/6/logs/FAIL__Clang__modules-header-sharing_m Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134222/new/ https:

[PATCH] D134222: [clang][deps] Report module map describing compiled module

This revision was automatically updated to reflect the committed changes. Closed by commit rG9dc0b1674841: [clang][deps] Report module map describing compiled module (authored by jansvoboda11). Herald added subscribers: Sanitizers, Enna1. Herald added a project: Sanitizers. Changed prior to commi

[PATCH] D134222: [clang][deps] Report module map describing compiled module

Bigcheese accepted this revision. Bigcheese added a comment. This revision is now accepted and ready to land. lgtm. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134222/new/ https://reviews.llvm.org/D134222

[PATCH] D134222: [clang][deps] Report module map describing compiled module

jansvoboda11 updated this revision to Diff 461431. jansvoboda11 added a comment. Implement suggestions Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134222/new/ https://reviews.llvm.org/D134222 Files: clang/include/clang/Basic/LangOptions.h cl

[PATCH] D134222: [clang][deps] Report module map describing compiled module

benlangmuir added inline comments. Comment at: clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp:194 +const FileEntry *CurrentModMap = +PP.getHeaderSearchInfo().getModuleMap().getModuleMapFileForUniquing( +CurrentModule); I think

[PATCH] D134222: [clang][deps] Report module map describing compiled module

jansvoboda11 created this revision. jansvoboda11 added reviewers: benlangmuir, Bigcheese. 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. This patch fixes compil