Author: Aiden Grossman
Date: 2025-11-23T20:41:08Z
New Revision: d40c8dccff70c0ffd4a6120334e4fcd472d2fd91

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

LOG: [Clang][OpenMP] Make test use clang_cc1 (#169233)

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).

Added: 
    

Modified: 
    clang/test/OpenMP/parallel_default_variableCategory_codegen.cpp

Removed: 
    


################################################################################
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


        
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to