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

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

No objection here, but I want to note that recomputing the range should be as simple as asking SCEV for the range of the new IV.  We'd basically just need a match for fptosi of sitofp, and then query the range of the input to the sitofp.  We should probably be doing that in addition to this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129140