[clang] [Clang][OpenMP] Make test use clang_cc1 (PR #169233)

via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 23 12:02:30 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

This test does not actually need to use the clang driver. Using the driver means that the environment plays much more into the tests results. We ran into a situation where the driver decided not to pass -fopenmp to the cc1 invocation, causing the test to fail.

This also makes the test more consistent with the other OpenMP tests and should make it slightly faster (no subprocess invocation).

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


1 Files Affected:

- (modified) clang/test/OpenMP/parallel_default_variableCategory_codegen.cpp (+1-1) 


``````````diff
diff --git a/clang/test/OpenMP/parallel_default_variableCategory_codegen.cpp b/clang/test/OpenMP/parallel_default_variableCategory_codegen.cpp
index e94d590933c85..8aa5855e25998 100644
--- a/clang/test/OpenMP/parallel_default_variableCategory_codegen.cpp
+++ b/clang/test/OpenMP/parallel_default_variableCategory_codegen.cpp
@@ -1,4 +1,4 @@
-// RUN: %clangxx -Xclang -verify -Wno-vla -fopenmp -fopenmp-version=60 -x c++ -S -emit-llvm %s -o - | FileCheck %s
+// RUN: %clang_cc1 -verify -fopenmp -fopenmp-version=60 -x c++ -emit-llvm %s -o - | FileCheck %s
 // expected-no-diagnostics
 #ifndef HEADER
 #define HEADER

``````````

</details>


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


More information about the cfe-commits mailing list