[llvm-branch-commits] [clang] clang: Simplify OpenMP triple adjustment (PR #189265)

[Joseph Huber via llvm-branch-commits]

https://github.com/jhuber6 approved this pull request.

Honestly, I wish we could just remove this altogether. This was only done because it was deemed too complicated for users to type this back before the new driver.
```
-fopenmp-targets=amdgcn-amd-amdhsa -Xopenmp-target=amdgcn-amd-amdhsa -march=1030
```
So this shortened it to
```
-fopenmp-targets=amdgcn -Xopenmp-target=amdgcn -march=1030
```
But now we can just do
```
--offload-arch=gfx1030
```
So it really doesn't need to exist, it's just an artifact that infers potentially incorrect information. That being said, I'm sure it's 'tested' somewhere, so it'd be a pain to remove.

https://github.com/llvm/llvm-project/pull/189265