[llvm] [AMDGPU] Fix flat load test to use addrspace 0 (PR #138306)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Sun May 4 00:05:14 PDT 2025
================
@@ -195,16 +195,16 @@ body: |
; GFX7-LABEL: name: load_flat_v2s32
; GFX7: liveins: $vgpr0_vgpr1
; GFX7-NEXT: {{ $}}
- ; GFX7-NEXT: [[COPY:%[0-9]+]]:vreg_64 = COPY $vgpr0_vgpr1
- ; GFX7-NEXT: [[FLAT_LOAD_DWORDX2_:%[0-9]+]]:vreg_64 = FLAT_LOAD_DWORDX2 [[COPY]], 0, 0, implicit $exec, implicit $flat_scr :: (load (<2 x s32>))
- ; GFX7-NEXT: $vgpr0_vgpr1 = COPY [[FLAT_LOAD_DWORDX2_]]
+ ; GFX7-NEXT: [[COPY:%[0-9]+]]:vgpr(p0) = COPY $vgpr0_vgpr1
+ ; GFX7-NEXT: [[LOAD:%[0-9]+]]:vgpr(<2 x s32>) = G_LOAD [[COPY]](p0) :: (load (<2 x s32>))
+ ; GFX7-NEXT: $vgpr0_vgpr1 = COPY [[LOAD]](<2 x s32>)
----------------
arsenm wrote:
I'm guessing it's because FlatLoadPat uses i64:$vaddr. I'm not sure how the pointer types and integer types interact exactly, but this seems like a bug that p1 would work but not p0. Does it work correctly if you change that to p0?
https://github.com/llvm/llvm-project/pull/138306
More information about the llvm-commits
mailing list