[PATCH] D104874: [AMDGPU] Add S_MOV_B64_IMM_PSEUDO for wide constants

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 29 15:54:08 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNCombineTupleInits.cpp:94
+
+    switch (I.getOperand(0).getSubReg()) {
+    default:
----------------
What happens if there are additional defs? I also don't see a dedicated MIR test for this?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104874/new/

https://reviews.llvm.org/D104874



More information about the llvm-commits mailing list