[PATCH] D109255: [NFC][OpenMP] Add fsyntax-only to driver tests

Jinsong Ji via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 3 12:40:51 PDT 2021


jsji created this revision.
jsji added reviewers: jdenny, ABataev, jdoerfert.
Herald added subscribers: guansong, yaxunl.
jsji requested review of this revision.
Herald added subscribers: cfe-commits, sstefan1.
Herald added a project: clang.

The test driver-fopenmp-extensions.c is failing on platforms that does
not use integrated-as. It can be reproduced using -fno-integrated-as on
Linux too.

bin/clang -c -Xclang -verify=omp -fopenmp      -fopenmp-extensions
-fno-openmp-extensions
../llvm-project/clang/test/OpenMP/driver-fopenmp-extensions.c
-fno-integrated-as
Assembler messages:
Error: can't open /tmp/driver-fopenmp-extensions-8fafe8.s for reading:
No such file or directory
clang-14: error: assembler command failed with exit code 1 (use -v to
see invocation)

The goal of this test is to verify syntax diags only,
so I think we should add -fsyntax-only to the test similar to other diag
tests.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D109255

Files:
  clang/test/OpenMP/driver-fopenmp-extensions.c


Index: clang/test/OpenMP/driver-fopenmp-extensions.c
===================================================================
--- clang/test/OpenMP/driver-fopenmp-extensions.c
+++ clang/test/OpenMP/driver-fopenmp-extensions.c
@@ -1,20 +1,20 @@
-// RUN: %clang -c -Xclang -verify=ompx -fopenmp %s
-// RUN: %clang -c -Xclang -verify=ompx -fopenmp-simd %s
+// RUN: %clang -c -fsyntax-only -Xclang -verify=ompx -fopenmp %s
+// RUN: %clang -c -fsyntax-only -Xclang -verify=ompx -fopenmp-simd %s
 
-// RUN: %clang -c -Xclang -verify=ompx -fopenmp -fopenmp-extensions %s
-// RUN: %clang -c -Xclang -verify=ompx -fopenmp-simd -fopenmp-extensions %s
+// RUN: %clang -c -fsyntax-only -Xclang -verify=ompx -fopenmp -fopenmp-extensions %s
+// RUN: %clang -c -fsyntax-only -Xclang -verify=ompx -fopenmp-simd -fopenmp-extensions %s
 
-// RUN: %clang -c -Xclang -verify=omp -fopenmp -fno-openmp-extensions %s
-// RUN: %clang -c -Xclang -verify=omp -fopenmp-simd -fno-openmp-extensions %s
+// RUN: %clang -c -fsyntax-only -Xclang -verify=omp -fopenmp -fno-openmp-extensions %s
+// RUN: %clang -c -fsyntax-only -Xclang -verify=omp -fopenmp-simd -fno-openmp-extensions %s
 
-// RUN: %clang -c -Xclang -verify=omp -fopenmp \
+// RUN: %clang -c -fsyntax-only -Xclang -verify=omp -fopenmp \
 // RUN:     -fopenmp-extensions -fno-openmp-extensions %s
-// RUN: %clang -c -Xclang -verify=omp -fopenmp-simd \
+// RUN: %clang -c -fsyntax-only -Xclang -verify=omp -fopenmp-simd \
 // RUN:     -fopenmp-extensions -fno-openmp-extensions %s
 
-// RUN: %clang -c -Xclang -verify=ompx -fopenmp \
+// RUN: %clang -c -fsyntax-only -Xclang -verify=ompx -fopenmp \
 // RUN:     -fno-openmp-extensions -fopenmp-extensions %s
-// RUN: %clang -c -Xclang -verify=ompx -fopenmp-simd \
+// RUN: %clang -c -fsyntax-only -Xclang -verify=ompx -fopenmp-simd \
 // RUN:     -fno-openmp-extensions -fopenmp-extensions %s
 
 void foo() {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D109255.370652.patch
Type: text/x-patch
Size: 1885 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210903/74e76b5d/attachment.bin>


More information about the cfe-commits mailing list