[clang] 35b1cfc - [Driver][Modules] Remove dependence on linking support from clang/test/Driver/modules.cpp

Daniel McIntosh via cfe-commits cfe-commits at lists.llvm.org
Tue May 31 07:34:02 PDT 2022


Author: Daniel McIntosh
Date: 2022-05-31T10:33:55-04:00
New Revision: 35b1cfc76f08faabf3f27c0bd054acc7c854a6ca

URL: https://github.com/llvm/llvm-project/commit/35b1cfc76f08faabf3f27c0bd054acc7c854a6ca
DIFF: https://github.com/llvm/llvm-project/commit/35b1cfc76f08faabf3f27c0bd054acc7c854a6ca.diff

LOG: [Driver][Modules] Remove dependence on linking support from clang/test/Driver/modules.cpp

The new tests in clang/test/Driver/modules.cpp added by D120540 will fail if the
toolchain getting tested doesn't support linking. This reduces the utility of
the test since we would like a failure of this test to reflect a problem with
modules. We should already have other tests that validate linking support.

Reviewed By: ChuanqiXu

Differential Revision: https://reviews.llvm.org/D126669

Added: 
    

Modified: 
    clang/test/Driver/modules.cpp

Removed: 
    


################################################################################
diff  --git a/clang/test/Driver/modules.cpp b/clang/test/Driver/modules.cpp
index 472b07c657992..f532b5b3c775f 100644
--- a/clang/test/Driver/modules.cpp
+++ b/clang/test/Driver/modules.cpp
@@ -76,11 +76,8 @@ FOO;
 
 // Check the independent use of -fcxx-modules
 //
-// RUN: %clang -fcxx-modules -std=c++17 -### %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX17-MODULES
-// CHECK-CXX17-MODULES: "-fcxx-modules"
-// RUN: %clang -fcxx-modules -std=c++14 -### %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX14-MODULES
-// CHECK-CXX14-MODULES: "-fcxx-modules"
-// RUN: %clang -fcxx-modules -std=c++11 -### %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX11-MODULES
-// CHECK-CXX11-MODULES: "-fcxx-modules"
-// RUN: %clang -fcxx-modules -std=c++03 -### %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX03-MODULES
-// CHECK-CXX03-MODULES: "-fcxx-modules"
+// RUN: %clang -fcxx-modules -std=c++17 -### -c %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX-MODULES
+// RUN: %clang -fcxx-modules -std=c++14 -### -c %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX-MODULES
+// RUN: %clang -fcxx-modules -std=c++11 -### -c %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX-MODULES
+// RUN: %clang -fcxx-modules -std=c++03 -### -c %s 2>&1 | FileCheck %s --check-prefix=CHECK-CXX-MODULES
+// CHECK-CXX-MODULES: "-fcxx-modules"


        


More information about the cfe-commits mailing list