[PATCH] D148796: [AMDGPU][GFX908] Add builtin support for global add atomic f16/f32

Thu Apr 20 12:13:21 PDT 2023

b-sumner added a comment.

In D148796#4284504 <https://reviews.llvm.org/D148796#4284504>, @rampitec wrote:

> We used to support it that way and decided just not doing it. It is very hard to explain why a supported atomic results in error. Someone who really needs it can use intrinsic.

I tend to agree.  This oddity is probably best handled with an intrinsic.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148796/new/

https://reviews.llvm.org/D148796