[PATCH] D137253: [CVP] Simplify comparisons without constant operand

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 3 06:16:59 PDT 2022


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

LGTM



================
Comment at: llvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp:351
   ++NumCmps;
   Constant *TorF = ConstantInt::get(Type::getInt1Ty(Cmp->getContext()), Result);
   Cmp->replaceAllUsesWith(TorF);
----------------
Should update this one to `CmpInst::makeCmpResultType()` too to future-proof it.


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

https://reviews.llvm.org/D137253



More information about the llvm-commits mailing list