[PATCH] D129140: [IndVars] Directly use integer induction for FPToSI of float induction.

[Florian Hahn via Phabricator via llvm-commits]

fhahn marked an inline comment as done.
fhahn added a comment.





================
Comment at: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:401
+      if (FPToSI && FPToSI->getType() == NewPHI->getType()) {
+        U.getUser()->replaceAllUsesWith(NewPHI);
+        continue;
----------------
nikic wrote:
> Add `FPToSI` to `DeadInsts`?
Done, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129140