[all-commits] [llvm/llvm-project] 76894c: clang/AMDGPU: Emit atomicrmw from ds_fadd builtins...

[Matt Arsenault via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 76894c5e6e20bfe8a30f7d8bdd39c41a7af54d65
      https://github.com/llvm/llvm-project/commit/76894c5e6e20bfe8a30f7d8bdd39c41a7af54d65
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2024-06-18 (Tue, 18 Jun 2024)

  Changed paths:
    M clang/lib/CodeGen/CGBuiltin.cpp
    M clang/test/CodeGenCUDA/builtins-amdgcn.cu
    M clang/test/CodeGenCUDA/builtins-spirv-amdgcn.cu
    M clang/test/CodeGenCUDA/builtins-unsafe-atomics-gfx90a.cu
    M clang/test/CodeGenCUDA/builtins-unsafe-atomics-spirv-amdgcn-gfx90a.cu
    M clang/test/CodeGenOpenCL/builtins-amdgcn-vi.cl
    M clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx12.cl
    M clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx8.cl
    M clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx90a.cl
    M clang/test/CodeGenOpenCL/builtins-fp-atomics-gfx940.cl
    M llvm/include/llvm/IR/IntrinsicsAMDGPU.td

  Log Message:
  -----------
  clang/AMDGPU: Emit atomicrmw from ds_fadd builtins (#95395)

We should have done this for the f32/f64 case a long time ago. Now that
codegen handles atomicrmw selection for the v2f16/v2bf16 case, start emitting
it instead.

This also does upgrade the behavior to respect a volatile qualified pointer,
which was previously ignored (for the cases that don't have an explicit
volatile argument).



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