Hooked up raw.buffer.atomic.fmin/max.f64
This instruction should be available on GFX6, GFX7, and GFX10.
It was implemented for GFX90a with a different name.
Added intrinsic def for image_atomic_fmin/fmax; the instruction
defs were already there.
Paths
| Differential D108208
[AMDGPU] Fix atomic float max/min intrinsics ClosedPublic Authored by Joe_Nash on Aug 17 2021, 8:12 AM.
Details Summary Hooked up raw.buffer.atomic.fmin/max.f64 Added intrinsic def for image_atomic_fmin/fmax; the instruction
Diff Detail
Event TimelineHerald added subscribers: foad, kerbowa, jfb and 7 others. · View Herald TranscriptAug 17 2021, 8:12 AM Comment Actions add globalisel tests. Apparently gisel does some checking of addrspaces Comment Actions
Not sure what you mean here. Are you just hitting broken addrspace(1) stores on SI? Comment Actions
Yes. This revision is now accepted and ready to land.Aug 17 2021, 3:28 PM Closed by commit rG9dbc968ed9fa: [AMDGPU] Fix atomic float max/min intrinsics (authored by Joe_Nash). · Explain WhyAug 18 2021, 11:29 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 366905 llvm/include/llvm/IR/IntrinsicsAMDGPU.td
llvm/lib/Target/AMDGPU/BUFInstructions.td
llvm/test/CodeGen/AMDGPU/fp-min-max-atomics.ll
|