[PATCH] D136007: [clang][modules][deps] System module maps might not be affecting

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 17 21:09:05 PDT 2022


jansvoboda11 added inline comments.


================
Comment at: clang/test/ClangScanDeps/modules-redefinition.m:37
+// RUN: %clang @%t/third.cc1.rsp
+// RUN: %clang @%t/first.cc1.rsp
----------------
Before this patch, this would fail with duplicate definition of module 'X'. It would be reached from the explicitly-provided `-fmodule-map-file=%t/zeroth/module.modulemap` and the implicitly discovered `%t/second/module.modulemap`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136007



More information about the cfe-commits mailing list