Does not handled atomics yet.
Diff Detail
Diff Detail
Event Timeline
Comment Actions
flakebi is right that some of the A16 codegen looks dodgy, but that may be due to legalization? I think it's okay to land this patch to get started.
llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.image.gather4.a16.dim.ll | ||
---|---|---|
77–79 | We're really missing any combines necessary to cleanup legalization |
STI.hasNSAEncoding()?