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

Wed Jul 6 19:16:44 PDT 2022

Allen added a comment.

In D129140#3630425 <https://reviews.llvm.org/D129140#3630425>, @reames wrote:

> 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.
>
> Edit: getSignedRange after calling computeSCEVAtScope that is.

hi @reames , out of interest , I tried to recomputing the range in https://reviews.llvm.org/D129191 as an addition patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129140