[PATCH] D127253: [AMDGPU] Use v_mad_u64_u32 for IMAD32
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 10 09:07:59 PDT 2022
rampitec added inline comments.
================
Comment at: llvm/test/CodeGen/AMDGPU/mad_u64_u32.ll:69
+; GFX9-NEXT: v_mov_b32_e32 v2, 0x12d687
+; GFX9-NEXT: v_mov_b32_e32 v3, 0
+; GFX9-NEXT: v_mad_u64_u32 v[0:1], s[0:1], v0, v1, v[2:3]
----------------
foad wrote:
> This mov is not required (since the high part of the 64-bit constant is undef) but I guess it is not worth the effort to try to eliminate it.
Yes, this would not be trivial.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127253/new/
https://reviews.llvm.org/D127253
More information about the llvm-commits
mailing list