[PATCH] D132939: [Peephole] rewrite INSERT_SUBREG to SUBREG_TO_REG if upper bits zero

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 8 18:46:37 PDT 2022


Allen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64MIPeepholeOpt.cpp:295
+          .add(MI.getOperand(3));
+  LLVM_DEBUG(dbgs() << MI << "  replace by:\n: " << *SubregMI << "\n");
+  MI.eraseFromParent();
----------------
MaskRay wrote:
> `SubregMI` caused a -Wunused-variable in `-DLLVM_ENABLE_ASSERTIONS=off` builds. I fixed it 
oh, yes, thanks vey much, I see your commit f9b59249757


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132939



More information about the llvm-commits mailing list