[clang] [Driver] Add `--` to some test clang-cl command lines. (PR #70055)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 24 08:37:20 PDT 2023
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-driver
Author: Simon Tatham (statham-arm)
<details>
<summary>Changes</summary>
If clang/test/Driver/cl-offload.cu is run on Unix in a directory whose absolute pathname starts with `/w`, such as the `/workspace` used by at least some Jenkins CI setups, then the file name on the clang command line is misinterpreted as some kind of MSVC warning-control option, and ignored by the catch-all `_SLASH_w` option in Options.td.
Other clang-cl tests take care to put a `--` before the input file name, to force clang to treat it as a filename even if it starts with a / and accidentally looks like a cl option. Do the same here.
---
Full diff: https://github.com/llvm/llvm-project/pull/70055.diff
1 Files Affected:
- (modified) clang/test/Driver/cl-offload.cu (+2-2)
``````````diff
diff --git a/clang/test/Driver/cl-offload.cu b/clang/test/Driver/cl-offload.cu
index 650c13da15b5b58..eaa4b58afa8878b 100644
--- a/clang/test/Driver/cl-offload.cu
+++ b/clang/test/Driver/cl-offload.cu
@@ -5,11 +5,11 @@
// RUN: %clang_cl -### -target x86_64-pc-windows-msvc --offload-arch=sm_35 -fgpu-rdc \
// RUN: --cuda-path=%S/Inputs/CUDA/usr/local/cuda \
-// RUN: /Wall -x cuda %s 2>&1 \
+// RUN: /Wall -x cuda -- %s 2>&1 \
// RUN: | FileCheck %s -check-prefix=CUDA
// RUN: %clang_cl -### -target x86_64-pc-windows-msvc --offload-arch=gfx1010 -fgpu-rdc --hip-link \
-// RUN: --rocm-path=%S/Inputs/rocm /Wall -x hip %s 2>&1 \
+// RUN: --rocm-path=%S/Inputs/rocm /Wall -x hip -- %s 2>&1 \
// RUN: | FileCheck %s -check-prefix=HIP
// CUDA: "-cc1" "-triple" "nvptx64-nvidia-cuda" "-aux-triple" "x86_64-pc-windows-msvc"
``````````
</details>
https://github.com/llvm/llvm-project/pull/70055
More information about the cfe-commits
mailing list