[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