[PATCH] D103547: Don't delete the module you're inspecting

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG13a9b2220f22: Don't delete the module you're inspecting (authored by beanz). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[PATCH] D103547: Don't delete the module you're inspecting

phosek accepted this revision. phosek added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103547/new/ https://reviews.llvm.org/D103547 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[PATCH] D103547: Don't delete the module you're inspecting

steven_wu accepted this revision. steven_wu 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/D103547/new/ https://reviews.llvm.org/D103547 _

[PATCH] D103547: Don't delete the module you're inspecting

beanz updated this revision to Diff 349350. beanz added a comment. Breaking out a separate test case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103547/new/ https://reviews.llvm.org/D103547 Files: clang/lib/Driver/Driver.cpp clang/test/Modu

[PATCH] D103547: Don't delete the module you're inspecting

steven_wu added a comment. Can you create a new test for clang Driver instead of rewrite the cc1 test? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103547/new/ https://reviews.llvm.org/D103547 ___ cfe-c

[PATCH] D103547: Don't delete the module you're inspecting

beanz created this revision. beanz added reviewers: steven_wu, arphaman. beanz requested review of this revision. Herald added a project: clang. Prior to this patch when you used `clang -module-file-info` clang would delete the module on completion because the module was treated as an output file