Fix bug when compiling CUDA code with -emit-llvm and -o.
authorSamuel Antao <sfantao@us.ibm.com>
Thu, 27 Oct 2016 00:53:34 +0000 (00:53 +0000)
committerSamuel Antao <sfantao@us.ibm.com>
Thu, 27 Oct 2016 00:53:34 +0000 (00:53 +0000)
In this case the device code is not injected into an host action and therefore the
user should get an error as -o can't be used when generating two outputs.

llvm-svn: 285263

clang/lib/Driver/Driver.cpp
clang/test/Driver/cuda-output-asm.cu

index f2ee1ca..4335bbf 100644 (file)
@@ -1611,6 +1611,11 @@ class OffloadingActionBuilder final {
 
         // We avoid creating host action in device-only mode.
         return CompileDeviceOnly ? ABRT_Ignore_Host : ABRT_Success;
+      } else if (CurPhase > phases::Backend) {
+        // If we are past the backend phase and still have a device action, we
+        // don't have to do anything as this action is already a device
+        // top-level action.
+        return ABRT_Success;
       }
 
       assert(CurPhase < phases::Backend && "Generating single CUDA "
index af62478..af066ce 100644 (file)
@@ -1,4 +1,4 @@
-// Tests CUDA compilation with -S.
+// Tests CUDA compilation with -S and -emit-llvm.
 
 // REQUIRES: clang-driver
 // REQUIRES: x86-registered-target
@@ -26,4 +26,6 @@
 // RUN: %clang -### -S -target x86_64-linux-gnu --cuda-device-only \
 // RUN:   --cuda-gpu-arch=sm_20 --cuda-gpu-arch=sm_30 -o foo.s %s 2>&1 \
 // RUN:   | FileCheck -check-prefix MULTIPLE-OUTPUT-FILES %s
+// RUN: %clang -### -emit-llvm -c -target x86_64-linux-gnu -o foo.s %s 2>&1 \
+// RUN:   | FileCheck -check-prefix MULTIPLE-OUTPUT-FILES %s
 // MULTIPLE-OUTPUT-FILES: error: cannot specify -o when generating multiple output files