[PATCH] D37914: [OpenMP] Don't throw cudalib not found error if only front-end is required.

Gheorghe-Teodor Bercea via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 18 11:57:42 PDT 2017


gtbercea added inline comments.


================
Comment at: test/Driver/openmp-offload-gpu.c:133
+/// Check that the flag is passed when -fopenmp-relocatable-target is used.
+// RUN:   %clang -### -c -S -fopenmp=libomp -fopenmp-targets=nvptx64-nvidia-cuda -Xopenmp-target -march=sm_99 -fopenmp-relocatable-target -save-temps -no-canonical-prefixes %s 2>&1 \
+// RUN:   | FileCheck -check-prefix=CHK-NOLIBDEVICE %s
----------------
tra wrote:
> I'd have separate runs -- one with -S and another one with -c.
> 
> That would cover openMP compilation changes. There should also be similar test to verify new behavior for CUDA compilation.
There is no new behaviour for CUDA actually, I only introduce this for OpenMP offloading.


Repository:
  rL LLVM

https://reviews.llvm.org/D37914





More information about the cfe-commits mailing list