[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