[PATCH] D129958: [IndVars] Eliminate redundant type cast with different sizes

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 07:34:25 PDT 2022


nikic added inline comments.


================
Comment at: llvm/test/Transforms/IndVarSimplify/floating-point-small-iv.ll:181
+
+define void @sitofp_fptoui_range_zext_postinc() {
+;
----------------
This isn't quite what I had in mind. By "postinc IV" I mean that `%inc.int` gets passed to sitofp, rather than `%iv.int`. I think your current code may result in a wrong insertion point for this case.


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

https://reviews.llvm.org/D129958



More information about the llvm-commits mailing list