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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to