[PATCH] D95455: [InstCombine] Preserve FMF for powi simplifications.

[Sanjay Patel via Phabricator via llvm-commits]

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

LGTM



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:887
   case Intrinsic::powi:
     if (ConstantInt *Power = dyn_cast<ConstantInt>(II->getArgOperand(1))) {
       // 0 and 1 are handled in instsimplify
----------------
It would be another patch, but if you're looking at powi code, it would be nice to update this with something like:
  match(II->getArgOperand, m_APInt(ExponentC))

That would allow us to also match vector powi calls in addition to the current scalar calls.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95455