[all-commits] [llvm/llvm-project] 272ce9: [Clang] Make OpenMP offloading consistently use th...

[Joseph Huber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 272ce90ed4814a2a5c87c78b5bcf24fc60b5703c
      https://github.com/llvm/llvm-project/commit/272ce90ed4814a2a5c87c78b5bcf24fc60b5703c
  Author: Joseph Huber <huberjn at outlook.com>
  Date:   2025-01-31 (Fri, 31 Jan 2025)

  Changed paths:
    M clang/lib/Driver/Driver.cpp
    M clang/lib/Driver/ToolChains/AMDGPUOpenMP.cpp
    M clang/lib/Driver/ToolChains/Cuda.cpp
    M clang/test/Driver/amdgpu-openmp-system-arch-fail.c
    M clang/test/Driver/amdgpu-openmp-toolchain.c
    M clang/test/Driver/openmp-offload-gpu.c
    M clang/test/Driver/openmp-offload-jit.c
    M clang/test/Driver/openmp-system-arch.c

  Log Message:
  -----------
  [Clang] Make OpenMP offloading consistently use the bound architecture (#125135)

Summary:
OpenMP was weirdly split between using the bound architecture from
`--offload-arch=` and the old `-march=` option which only worked for
single jobs. This patch removes that special handling. The main benefit
here is that we can now use `getToolchainArgs` without it throwing an
error.

I'm assuming SYCL doesn't care about this because they don't use an
architecture.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications