[PATCH] D158495: [PowerPC] Try to use less instructions to materialize 64-bit constant when High32=Low32

[Esme Yi via Phabricator via llvm-commits]

Esme added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp:1181-1187
+    else {
+      InstCnt = 3;
+      Result =
+          CurDAG->getMachineNode(PPC::LIS8, dl, MVT::i64, getI32Imm(ImmHi16));
+      Result = CurDAG->getMachineNode(PPC::ORI8, dl, MVT::i64,
+                                      SDValue(Result, 0), getI32Imm(ImmLo16));
+    }
----------------
qiucf wrote:
> Can we remove this part? I think it's already been handled.
Do you mean the case where 3 instructions are needed?
If we remove it, we will use 4 instructions for constants like 0xFFF7C000FFF7C000.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158495