[clang] Add tests for driver to propagate module map flags even without -fmod… (PR #75827)

via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 18 09:08:30 PST 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-driver

Author: Walter Lee (googlewalt)

<details>
<summary>Changes</summary>

…ules

We use this to verify properties of our build graph, even when modules are off.  Xcode 14.3.1 seems to have dropped these flags so we are creating a unit test to reproduce the issue.

---
Full diff: https://github.com/llvm/llvm-project/pull/75827.diff


1 Files Affected:

- (modified) clang/test/Driver/modules.m (+8) 


``````````diff
diff --git a/clang/test/Driver/modules.m b/clang/test/Driver/modules.m
index 9eb3569805569a..20bbcf40512300 100644
--- a/clang/test/Driver/modules.m
+++ b/clang/test/Driver/modules.m
@@ -57,6 +57,14 @@
 // CHECK-MODULE-MAP-FILES: "-fmodule-map-file=foo.map"
 // CHECK-MODULE-MAP-FILES: "-fmodule-map-file=bar.map"
 
+// RUN: %clang -fno-modules -fmodule-name=foo -c -### %s 2>&1 | FileCheck -check-prefix=CHECK-PROPAGATE-MODULE-NAME %s
+//
+// CHECK-PROPAGATE-MODULE-NAME: -fmodule-name=foo
+
+// RUN: %clang -fno-modules -fmodule-map-file=foo.map -c -### %s 2>&1 | FileCheck -check-prefix=CHECK-PROPAGATE-MODULE-MAPS %s
+//
+// CHECK-PROPAGATE-MODULE-MAPS: -fmodule-map-file=foo.map
+
 // RUN: %clang -fmodules -fbuiltin-module-map -### %s 2>&1 | FileCheck -check-prefix=CHECK-BUILTIN-MODULE-MAP %s
 // CHECK-BUILTIN-MODULE-MAP: "-fmodules"
 // CHECK-BUILTIN-MODULE-MAP: "-fmodule-map-file={{.*}}include{{/|\\\\}}module.modulemap"

``````````

</details>


https://github.com/llvm/llvm-project/pull/75827


More information about the cfe-commits mailing list