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