[PATCH] D81723: [PowerPC] fold addi's imm operand to its imm form consumer's displacement

ChenZheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 23 02:36:30 PDT 2020


shchenz marked 2 inline comments as done.
shchenz added a comment.

Thanks for your review.



================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.cpp:3906
+  // isImmElgibleForForwarding.
+  int64_t Imm = ImmOperandMI.getImm();
+  if (!isImmElgibleForForwarding(*ImmMO, DefMI, III, Imm, Imm))
----------------
steven.zhang wrote:
> Don't use the same variable to avoid confusion.
Will update in the commit patch


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81723





More information about the llvm-commits mailing list