[PATCH] D129756: [IndVars] Directly use unsigned integer induction for FPToUI/FPToSI of float induction

[Allen zhong via Phabricator via llvm-commits]

Allen added inline comments.


================
Comment at: llvm/test/Transforms/IndVarSimplify/floating-point-small-iv.ll:184
+; https://godbolt.org/z/51MrqYjEf
+define void @uitofp_fptoui_range () {
+; CHECK-LABEL: @uitofp_fptoui_range(
----------------
nikic wrote:
> We should be testing negative ranges as well -- in this case, they would actually be not legal.
for unsigned type induction, negative ranges  will be determined true at very early pass, so we don't need check it ?
 https://godbolt.org/z/rerdG58cj


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

https://reviews.llvm.org/D129756