[clang] [CLANG][DRIVER] Resolve Argument Visibility and Duplication Issue for Cuda ToolChain (PR #86807)

2024-03-27 Thread Jefferson Le Quellec via cfe-commits 

https://github.com/jle-quel created 
https://github.com/llvm/llvm-project/pull/86807

# Description

This PR makes the argument `-Xcuda-ptxas` visible to the driver in cl-mode.

Furthermore, it has been noticed that the arguments are being passed twice to 
`ptxas`.
This also has been fixed by filtering out the arguments before appending them 
to the new `DAL` created by `CudaToolChain::TranslateArgs`.

>From e28fab463b54192b8e8440aa8c349c2d42bb949a Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Wed, 27 Mar 2024 15:25:16 +0100
Subject: [PATCH 1/3] Make the argument -Xcuda-ptxas visible to the driver in
 cl-mode

---
 clang/include/clang/Driver/Options.td | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/clang/include/clang/Driver/Options.td 
b/clang/include/clang/Driver/Options.td
index 29066ea14280c2..a5e132dc48a3ef 100644
--- a/clang/include/clang/Driver/Options.td
+++ b/clang/include/clang/Driver/Options.td
@@ -1001,7 +1001,8 @@ def : Joined<["-"], "Xclang=">, Group,
 def Xcuda_fatbinary : Separate<["-"], "Xcuda-fatbinary">,
   HelpText<"Pass  to fatbinary invocation">, MetaVarName<"">;
 def Xcuda_ptxas : Separate<["-"], "Xcuda-ptxas">,
-  HelpText<"Pass  to the ptxas assembler">, MetaVarName<"">;
+  HelpText<"Pass  to the ptxas assembler">, MetaVarName<"">,
+  Visibility<[ClangOption, CLOption]>;
 def Xopenmp_target : Separate<["-"], "Xopenmp-target">, 
Group,
   HelpText<"Pass  to the target offloading toolchain.">, 
MetaVarName<"">;
 def Xopenmp_target_EQ : JoinedAndSeparate<["-"], "Xopenmp-target=">, 
Group,

>From be1fd093d9c3f2553cefd061d476d052848b853a Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Wed, 27 Mar 2024 15:26:52 +0100
Subject: [PATCH 2/3] Make sure arguments are not duplicated in new DAL

---
 clang/lib/Driver/ToolChains/Cuda.cpp | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/clang/lib/Driver/ToolChains/Cuda.cpp 
b/clang/lib/Driver/ToolChains/Cuda.cpp
index 5f0b516e1a1a08..6634e6d818b33e 100644
--- a/clang/lib/Driver/ToolChains/Cuda.cpp
+++ b/clang/lib/Driver/ToolChains/Cuda.cpp
@@ -990,7 +990,10 @@ CudaToolChain::TranslateArgs(const 
llvm::opt::DerivedArgList &Args,
   }
 
   for (Arg *A : Args) {
-DAL->append(A);
+// Make sure flags are not duplicated.
+if (!llvm::is_contained(*DAL, A)) {
+  DAL->append(A);
+}
   }
 
   if (!BoundArch.empty()) {

>From fe2ab25ac7efc10851ca3aaa51caecc80f9000d4 Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Wed, 27 Mar 2024 15:27:33 +0100
Subject: [PATCH 3/3] Add visibility and duplication tests

---
 clang/test/Driver/cuda-external-tools.cu | 8 
 1 file changed, 8 insertions(+)

diff --git a/clang/test/Driver/cuda-external-tools.cu 
b/clang/test/Driver/cuda-external-tools.cu
index 946e144fce38fb..d9564d026b4faa 100644
--- a/clang/test/Driver/cuda-external-tools.cu
+++ b/clang/test/Driver/cuda-external-tools.cu
@@ -86,6 +86,12 @@
 // RUN:   -Xcuda-fatbinary -bar1 -Xcuda-ptxas -foo2 -Xcuda-fatbinary -bar2 %s 
2>&1 \
 // RUN: | FileCheck -check-prefixes=CHECK,SM35,PTXAS-EXTRA,FATBINARY-EXTRA %s
 
+// Check -Xcuda-ptxas with clang-cl
+// RUN: %clang_cl -### -c -Xcuda-ptxas -foo1 \
+// RUN:   --offload-arch=sm_35 --cuda-path=%S/Inputs/CUDA/usr/local/cuda \
+// RUN:   -Xcuda-ptxas -foo2 %s 2>&1 \
+// RUN: | FileCheck -check-prefixes=CHECK,SM35,PTXAS-EXTRA %s
+
 // MacOS spot-checks
 // RUN: %clang -### --target=x86_64-apple-macosx -O0 -c %s 2>&1 \
 // RUN:   --offload-arch=sm_35 --cuda-path=%S/Inputs/CUDA/usr/local/cuda \
@@ -140,6 +146,8 @@
 // CHECK-SAME: "[[PTXFILE]]"
 // PTXAS-EXTRA-SAME: "-foo1"
 // PTXAS-EXTRA-SAME: "-foo2"
+// CHECK-NOT: "-foo1"
+// CHECK-NOT: "-foo2"
 // RDC-SAME: "-c"
 // CHECK-NOT: "-c"
 

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

[clang] [Driver][OpenMP] Fix OpenMP target-toolchain-option parser (PR #115375)

2024-11-07 Thread Jefferson Le Quellec via cfe-commits 

https://github.com/jle-quel created 
https://github.com/llvm/llvm-project/pull/115375

## Description

This PR fixes a segmentation fault that occurs when passing options requiring 
arguments via `-Xopenmp-target=`. The issue was that the function 
`Driver::getOffloadArchs` did not properly parse the extracted option, but 
instead assumed   it was valid, leading to a crash when incomplete arguments 
were provided.

## Backtrace

```sh
llvm-project/build/bin/clang++ main.cpp -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu 
-Xopenmp-target=powerpc64le-ibm-linux-gnu -o 
PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and 
include the crash backtrace, preprocessed source, and associated run script.
Stack dump:
0.  Program arguments: llvm-project/build/bin/clang++ main.cpp 
-fopenmp=libomp -fopenmp-targets=powerpc64le-ibm-linux-gnu 
-Xopenmp-target=powerpc64le-ibm-linux-gnu -o
1.  Compilation construction
2.  Building compilation actions
 #0 0x562fb21c363b llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) 
(llvm-project/build/bin/clang+++0x392f63b)
 #1 0x562fb21c0e3c SignalHandler(int) Signals.cpp:0:0
 #2 0x7fcbf6c81420 __restore_rt 
(/lib/x86_64-linux-gnu/libpthread.so.0+0x14420)
 #3 0x562fb1fa5d70 llvm::opt::Option::matches(llvm::opt::OptSpecifier) 
const (llvm-project/build/bin/clang+++0x3711d70)
 #4 0x562fb2a78e7d 
clang::driver::Driver::getOffloadArchs(clang::driver::Compilation&, 
llvm::opt::DerivedArgList const&, clang::driver::Action::OffloadKind, 
clang::driver::ToolChain const*, bool) const 
(llvm-project/build/bin/clang+++0x41e4e7d)
 #5 0x562fb2a7a9aa 
clang::driver::Driver::BuildOffloadingActions(clang::driver::Compilation&, 
llvm::opt::DerivedArgList&, std::pair const&, clang::driver::Action*) const (.part.1164) Driver.cpp:0:0
 #6 0x562fb2a7c093 
clang::driver::Driver::BuildActions(clang::driver::Compilation&, 
llvm::opt::DerivedArgList&, 
llvm::SmallVector, 
16u> const&, llvm::SmallVector&) const 
(llvm-project/build/bin/clang+++0x41e8093)
 #7 0x562fb2a8395d 
clang::driver::Driver::BuildCompilation(llvm::ArrayRef) 
(llvm-project/build/bin/clang+++0x41ef95d)
 #8 0x562faf92684c clang_main(int, char**, llvm::ToolContext const&) 
(llvm-project/build/bin/clang+++0x109284c)
 #9 0x562faf826cc6 main (llvm-project/build/bin/clang+++0xf92cc6)
#10 0x7fcbf6699083 __libc_start_main 
/build/glibc-LcI20x/glibc-2.31/csu/../csu/libc-start.c:342:3
#11 0x562faf923a5e _start (llvm-project/build/bin/clang+++0x108fa5e)
[1]2628042 segmentation fault (core dumped)   main.cpp -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu  -o
```

>From b85cc64cee1ac22f27c648ca8d892b40804f6d98 Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Thu, 7 Nov 2024 22:38:26 +0100
Subject: [PATCH] Fix OpenMP target-toolchain-option parser

---
 clang/lib/Driver/Driver.cpp|  6 ++
 clang/lib/Driver/ToolChain.cpp |  6 --
 clang/test/Driver/openmp-offload.c | 10 --
 3 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp
index 93e85f7dffe35a..a7e985195a3051 100644
--- a/clang/lib/Driver/Driver.cpp
+++ b/clang/lib/Driver/Driver.cpp
@@ -4528,7 +4528,13 @@ Driver::getOffloadArchs(Compilation &C, const 
llvm::opt::DerivedArgList &Args,
 ToolChain::getOpenMPTriple(Arg->getValue(0)) == TC->getTriple()) {
   Arg->claim();
   unsigned Index = Args.getBaseArgs().MakeIndex(Arg->getValue(1));
+  unsigned Prev = Index;
   ExtractedArg = getOpts().ParseOneArg(Args, Index);
+  if (!ExtractedArg || Index > Prev + 1) {
+TC->getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
+<< Arg->getAsString(Args);
+continue;
+  }
   Arg = ExtractedArg.get();
 }
 
diff --git a/clang/lib/Driver/ToolChain.cpp b/clang/lib/Driver/ToolChain.cpp
index 646dbc0faf5a9f..188d2dc7e03e25 100644
--- a/clang/lib/Driver/ToolChain.cpp
+++ b/clang/lib/Driver/ToolChain.cpp
@@ -1598,8 +1598,10 @@ llvm::opt::DerivedArgList 
*ToolChain::TranslateOpenMPTargetArgs(
 Prev = Index;
 std::unique_ptr XOpenMPTargetArg(Opts.ParseOneArg(Args, Index));
 if (!XOpenMPTargetArg || Index > Prev + 1) {
-  getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
-  << A->getAsString(Args);
+  if (!A->isClaimed()) {
+getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
+<< A->getAsString(Args);
+  }
   continue;
 }
 if (XOpenMPTargetNoTriple && XOpenMPTargetArg &&
diff --git a/clang/test/Driver/openmp-offload.c 
b/clang/test/Driver/openmp-offload.c
index eaed0d66df75cb..8c787cbc695335 100644
--- a/clang/test/Driver/openmp-offload.c
+++ b/clang/test/Driver/openmp-offload.c
@@ -84,9 +84,15 @@
 
 /// Check -Xopenmp-target triggers error when an option requiring arguments is 
passed to it.
 // RUN:   not %clang -### -no-canonical-prefixes -

[clang] [Driver][OpenMP] Fix OpenMP target-toolchain-option parser (PR #115375)

2024-11-29 Thread Jefferson Le Quellec via cfe-commits 

jle-quel wrote:

Ping @alexey-bataev @MaskRay 

https://github.com/llvm/llvm-project/pull/115375
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver][OpenMP] Fix OpenMP target-toolchain-option parser (PR #115375)

2024-11-18 Thread Jefferson Le Quellec via cfe-commits 

https://github.com/jle-quel updated 
https://github.com/llvm/llvm-project/pull/115375

>From b85cc64cee1ac22f27c648ca8d892b40804f6d98 Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Thu, 7 Nov 2024 22:38:26 +0100
Subject: [PATCH] Fix OpenMP target-toolchain-option parser

---
 clang/lib/Driver/Driver.cpp|  6 ++
 clang/lib/Driver/ToolChain.cpp |  6 --
 clang/test/Driver/openmp-offload.c | 10 --
 3 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp
index 93e85f7dffe35a..a7e985195a3051 100644
--- a/clang/lib/Driver/Driver.cpp
+++ b/clang/lib/Driver/Driver.cpp
@@ -4528,7 +4528,13 @@ Driver::getOffloadArchs(Compilation &C, const 
llvm::opt::DerivedArgList &Args,
 ToolChain::getOpenMPTriple(Arg->getValue(0)) == TC->getTriple()) {
   Arg->claim();
   unsigned Index = Args.getBaseArgs().MakeIndex(Arg->getValue(1));
+  unsigned Prev = Index;
   ExtractedArg = getOpts().ParseOneArg(Args, Index);
+  if (!ExtractedArg || Index > Prev + 1) {
+TC->getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
+<< Arg->getAsString(Args);
+continue;
+  }
   Arg = ExtractedArg.get();
 }
 
diff --git a/clang/lib/Driver/ToolChain.cpp b/clang/lib/Driver/ToolChain.cpp
index 646dbc0faf5a9f..188d2dc7e03e25 100644
--- a/clang/lib/Driver/ToolChain.cpp
+++ b/clang/lib/Driver/ToolChain.cpp
@@ -1598,8 +1598,10 @@ llvm::opt::DerivedArgList 
*ToolChain::TranslateOpenMPTargetArgs(
 Prev = Index;
 std::unique_ptr XOpenMPTargetArg(Opts.ParseOneArg(Args, Index));
 if (!XOpenMPTargetArg || Index > Prev + 1) {
-  getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
-  << A->getAsString(Args);
+  if (!A->isClaimed()) {
+getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
+<< A->getAsString(Args);
+  }
   continue;
 }
 if (XOpenMPTargetNoTriple && XOpenMPTargetArg &&
diff --git a/clang/test/Driver/openmp-offload.c 
b/clang/test/Driver/openmp-offload.c
index eaed0d66df75cb..8c787cbc695335 100644
--- a/clang/test/Driver/openmp-offload.c
+++ b/clang/test/Driver/openmp-offload.c
@@ -84,9 +84,15 @@
 
 /// Check -Xopenmp-target triggers error when an option requiring arguments is 
passed to it.
 // RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu -Xopenmp-target -Xopenmp-target 
-mcpu=pwr8 %s 2>&1 \
-// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR %s
+// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_0 %s
 
-// CHK-FOPENMP-TARGET-NESTED-ERROR: clang{{.*}} error: invalid -Xopenmp-target 
argument: '-Xopenmp-target -Xopenmp-target', options requiring arguments are 
unsupported
+// CHK-FOPENMP-TARGET-NESTED-ERROR_0: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target -Xopenmp-target', options requiring 
arguments are unsupported
+
+/// Check -Xopenmp-target= triggers error when an option requiring arguments 
is passed to it.
+// RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu 
-Xopenmp-target=powerpc64le-ibm-linux-gnu -Xopenmp-target -mcpu=pwr8 %s 2>&1 \
+// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_1 %s
+
+// CHK-FOPENMP-TARGET-NESTED-ERROR_1: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target=powerpc64le-ibm-linux-gnu 
-Xopenmp-target', options requiring arguments are unsupported
 
 /// ###
 

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

[clang] [Driver][OpenMP] Fix OpenMP target-toolchain-option parser (PR #115375)

2024-12-02 Thread Jefferson Le Quellec via cfe-commits 

https://github.com/jle-quel updated 
https://github.com/llvm/llvm-project/pull/115375

>From b85cc64cee1ac22f27c648ca8d892b40804f6d98 Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Thu, 7 Nov 2024 22:38:26 +0100
Subject: [PATCH 1/2] Fix OpenMP target-toolchain-option parser

---
 clang/lib/Driver/Driver.cpp|  6 ++
 clang/lib/Driver/ToolChain.cpp |  6 --
 clang/test/Driver/openmp-offload.c | 10 --
 3 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp
index 93e85f7dffe35a..a7e985195a3051 100644
--- a/clang/lib/Driver/Driver.cpp
+++ b/clang/lib/Driver/Driver.cpp
@@ -4528,7 +4528,13 @@ Driver::getOffloadArchs(Compilation &C, const 
llvm::opt::DerivedArgList &Args,
 ToolChain::getOpenMPTriple(Arg->getValue(0)) == TC->getTriple()) {
   Arg->claim();
   unsigned Index = Args.getBaseArgs().MakeIndex(Arg->getValue(1));
+  unsigned Prev = Index;
   ExtractedArg = getOpts().ParseOneArg(Args, Index);
+  if (!ExtractedArg || Index > Prev + 1) {
+TC->getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
+<< Arg->getAsString(Args);
+continue;
+  }
   Arg = ExtractedArg.get();
 }
 
diff --git a/clang/lib/Driver/ToolChain.cpp b/clang/lib/Driver/ToolChain.cpp
index 646dbc0faf5a9f..188d2dc7e03e25 100644
--- a/clang/lib/Driver/ToolChain.cpp
+++ b/clang/lib/Driver/ToolChain.cpp
@@ -1598,8 +1598,10 @@ llvm::opt::DerivedArgList 
*ToolChain::TranslateOpenMPTargetArgs(
 Prev = Index;
 std::unique_ptr XOpenMPTargetArg(Opts.ParseOneArg(Args, Index));
 if (!XOpenMPTargetArg || Index > Prev + 1) {
-  getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
-  << A->getAsString(Args);
+  if (!A->isClaimed()) {
+getDriver().Diag(diag::err_drv_invalid_Xopenmp_target_with_args)
+<< A->getAsString(Args);
+  }
   continue;
 }
 if (XOpenMPTargetNoTriple && XOpenMPTargetArg &&
diff --git a/clang/test/Driver/openmp-offload.c 
b/clang/test/Driver/openmp-offload.c
index eaed0d66df75cb..8c787cbc695335 100644
--- a/clang/test/Driver/openmp-offload.c
+++ b/clang/test/Driver/openmp-offload.c
@@ -84,9 +84,15 @@
 
 /// Check -Xopenmp-target triggers error when an option requiring arguments is 
passed to it.
 // RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu -Xopenmp-target -Xopenmp-target 
-mcpu=pwr8 %s 2>&1 \
-// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR %s
+// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_0 %s
 
-// CHK-FOPENMP-TARGET-NESTED-ERROR: clang{{.*}} error: invalid -Xopenmp-target 
argument: '-Xopenmp-target -Xopenmp-target', options requiring arguments are 
unsupported
+// CHK-FOPENMP-TARGET-NESTED-ERROR_0: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target -Xopenmp-target', options requiring 
arguments are unsupported
+
+/// Check -Xopenmp-target= triggers error when an option requiring arguments 
is passed to it.
+// RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu 
-Xopenmp-target=powerpc64le-ibm-linux-gnu -Xopenmp-target -mcpu=pwr8 %s 2>&1 \
+// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_1 %s
+
+// CHK-FOPENMP-TARGET-NESTED-ERROR_1: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target=powerpc64le-ibm-linux-gnu 
-Xopenmp-target', options requiring arguments are unsupported
 
 /// ###
 

>From 6d897f13c237ad174680973751c272c1b459487f Mon Sep 17 00:00:00 2001
From: Jefferson Le Quellec 
Date: Mon, 2 Dec 2024 16:32:32 +0100
Subject: [PATCH 2/2] Update tests to omit the string before 'error'

---
 clang/test/Driver/openmp-offload.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/clang/test/Driver/openmp-offload.c 
b/clang/test/Driver/openmp-offload.c
index 8c787cbc695335..caedc223a5c76f 100644
--- a/clang/test/Driver/openmp-offload.c
+++ b/clang/test/Driver/openmp-offload.c
@@ -86,13 +86,13 @@
 // RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu -Xopenmp-target -Xopenmp-target 
-mcpu=pwr8 %s 2>&1 \
 // RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_0 %s
 
-// CHK-FOPENMP-TARGET-NESTED-ERROR_0: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target -Xopenmp-target', options requiring 
arguments are unsupported
+// CHK-FOPENMP-TARGET-NESTED-ERROR_0: error: invalid -Xopenmp-target argument: 
'-Xopenmp-target -Xopenmp-target', options requiring arguments are unsupported
 
 /// Check -Xopenmp-target= triggers error when an option requiring arguments 
is passed to it.
 // RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu 
-Xopenmp-target

[clang] [Driver][OpenMP] Fix OpenMP target-toolchain-option parser (PR #115375)

2024-12-02 Thread Jefferson Le Quellec via cfe-commits 


@@ -84,9 +84,15 @@
 
 /// Check -Xopenmp-target triggers error when an option requiring arguments is 
passed to it.
 // RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu -Xopenmp-target -Xopenmp-target 
-mcpu=pwr8 %s 2>&1 \
-// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR %s
+// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_0 %s
 
-// CHK-FOPENMP-TARGET-NESTED-ERROR: clang{{.*}} error: invalid -Xopenmp-target 
argument: '-Xopenmp-target -Xopenmp-target', options requiring arguments are 
unsupported
+// CHK-FOPENMP-TARGET-NESTED-ERROR_0: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target -Xopenmp-target', options requiring 
arguments are unsupported
+
+/// Check -Xopenmp-target= triggers error when an option requiring arguments 
is passed to it.
+// RUN:   not %clang -### -no-canonical-prefixes -fopenmp=libomp 
-fopenmp-targets=powerpc64le-ibm-linux-gnu 
-Xopenmp-target=powerpc64le-ibm-linux-gnu -Xopenmp-target -mcpu=pwr8 %s 2>&1 \
+// RUN:   | FileCheck -check-prefix=CHK-FOPENMP-TARGET-NESTED-ERROR_1 %s
+
+// CHK-FOPENMP-TARGET-NESTED-ERROR_1: clang{{.*}} error: invalid 
-Xopenmp-target argument: '-Xopenmp-target=powerpc64le-ibm-linux-gnu 
-Xopenmp-target', options requiring arguments are unsupported

jle-quel wrote:

Updated :+1: 

https://github.com/llvm/llvm-project/pull/115375
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver][OpenMP] Fix OpenMP target-toolchain-option parser (PR #115375)

2024-12-06 Thread Jefferson Le Quellec via cfe-commits 

jle-quel wrote:

@MaskRay @alexey-bataev, if everything looks good now, could we go ahead and 
merge this PR? Thanks for your time reviewing!

https://github.com/llvm/llvm-project/pull/115375
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits