[llvm] [AMDGPU] Introduce orderign parameter to atomic intrinsics and introduce new llvm.amdgcn.image.atomic.load intrinsic. (PR #73613)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 22 05:43:55 PST 2023
================
@@ -1154,6 +1154,8 @@ class TargetLoweringBase {
MaybeAlign align = Align(1); // alignment
MachineMemOperand::Flags flags = MachineMemOperand::MONone;
+
+ AtomicOrdering ordering = AtomicOrdering::NotAtomic;
----------------
sstipanovic wrote:
I started refactoring this to add `MachineMemOperand` field, but half way through it seemed to me that the code was more complex due to that change. It would require to construct MMO in multiple places in `getTgtMemIntrinsic` which seemed unnecessary to me. I think we now have all the necessary fields to construct MMO. What do you think?
https://github.com/llvm/llvm-project/pull/73613
More information about the llvm-commits
mailing list