[PATCH] D152614: [AMDGPU] Fix VGPR VOPD banks for src2
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 10 03:11:37 PDT 2023
rampitec added inline comments.
================
Comment at: llvm/test/MC/AMDGPU/gfx11_asm_vopd_err.s:281
-v_dual_fmamk_f32 v6, v1, 0xaf123456, v3 :: v_dual_fmamk_f32 v5, v2, 0xaf123456, v5
// GFX11: :[[@LINE-1]]:{{[0-9]+}}: error: src2 operands must use different VGPR banks
----------------
rampitec wrote:
> But then even a manually written tests do not save against a simple miscalculation. These registers are only 2 steps apart. There are 4 banks. One must add 4 to get a bank conflict. 3+4=7, not 5. How was this test done?
There is no bank conflict between v3 and v5, I am sorry.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152614/new/
https://reviews.llvm.org/D152614
More information about the llvm-commits
mailing list