From 25c6a22e4389dc5f40660b7c6c1c7ccb6759c33c Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Thu, 28 May 2015 21:20:14 +0000 Subject: [PATCH] [omp] Loosen the driver test enough so that overriding the defaults works well for folks. This isn't terribly clean (sadly) but after chatting with both Eric and Richard, nothing cleaner really emerged. The clean way of doing this is a *lot* of work for extremely little benefit here. llvm-svn: 238500 --- clang/test/Driver/fopenmp.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/clang/test/Driver/fopenmp.c b/clang/test/Driver/fopenmp.c index c7eec5fc0c84..36c856d8820e 100644 --- a/clang/test/Driver/fopenmp.c +++ b/clang/test/Driver/fopenmp.c @@ -1,4 +1,3 @@ -// RUN: %clang -target x86_64-linux-gnu -fopenmp -c %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-CC1-NO-OPENMP // RUN: %clang -target x86_64-linux-gnu -fopenmp=libomp -c %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-CC1-OPENMP // RUN: %clang -target x86_64-linux-gnu -fopenmp=libgomp -c %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-CC1-NO-OPENMP // RUN: %clang -target x86_64-linux-gnu -fopenmp=libiomp5 -c %s -### 2>&1 | FileCheck %s --check-prefix=CHECK-CC1-OPENMP @@ -9,7 +8,6 @@ // CHECK-CC1-NO-OPENMP: "-cc1" // CHECK-CC1-NO-OPENMP-NOT: "-fopenmp" // -// RUN: %clang -target x86_64-linux-gnu -fopenmp %s -o %t -### 2>&1 | FileCheck %s --check-prefix=CHECK-LD-GOMP // RUN: %clang -target x86_64-linux-gnu -fopenmp=libomp %s -o %t -### 2>&1 | FileCheck %s --check-prefix=CHECK-LD-OMP // RUN: %clang -target x86_64-linux-gnu -fopenmp=libgomp %s -o %t -### 2>&1 | FileCheck %s --check-prefix=CHECK-LD-GOMP // RUN: %clang -target x86_64-linux-gnu -fopenmp=libiomp5 %s -o %t -### 2>&1 | FileCheck %s --check-prefix=CHECK-LD-IOMP5 @@ -22,3 +20,14 @@ // // CHECK-LD-IOMP5: "{{.*}}ld{{(.exe)?}}" // CHECK-LD-IOMP5: "-liomp5" +// +// We'd like to check that the default is sane, but until we have the ability +// to *always* semantically analyze OpenMP without always generating runtime +// calls (in the event of an unsupported runtime), we don't have a good way to +// test the CC1 invocation. Instead, just ensure we do eventually link *some* +// OpenMP runtime. +// +// RUN: %clang -target x86_64-linux-gnu -fopenmp %s -o %t -### 2>&1 | FileCheck %s --check-prefix=CHECK-LD-ANY +// +// CHECK-LD-ANY: "{{.*}}ld{{(.exe)?}}" +// CHECK-LD-ANY: "-l{{(omp|gomp|iomp5)}}"