[PATCH] D60413: [BDCE] SExt -> ZExt when no sign bits is used and instruction has multiple uses

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 9 09:46:29 PDT 2020


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: llvm/lib/Transforms/Scalar/BDCE.cpp:125
+        IRBuilder<> Builder(SE);
+        I.replaceAllUsesWith(Builder.CreateZExt(SE->getOperand(0), DstTy));
+        Worklist.push_back(SE);
----------------
Please pass `SE->getName()` here to preserve the instruction name.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60413





More information about the llvm-commits mailing list