[PATCH] D143662: [AMDGPU] Refactor multiclass FLAT_Atomic_Pseudo. NFC.

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/FLATInstructions.td:469
 
-multiclass FLAT_Atomic_Pseudo<
+multiclass FLAT_Atomic_Pseudo_NO_RTN<
   string opName,
----------------
rampitec wrote:
> This can be just a class now?
Yes, and I did try that, but I preferred the consistency of having everything here be a multiclass. Note that both `FLAT_Global_Atomic_Pseudo_NO_RTN` and `FLAT_Global_Atomic_Pseudo_RTN` have to be multiclasses because they have to define additional _SADDR variants.

Do you feel strongly that it should be a class?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143662