[PATCH] D119261: [DependenceAnalysis][PR52170] Conservative crash on overflowed loop backedge

Bardia Mahjour via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 8 10:31:40 PST 2022


bmahjour added inline comments.


================
Comment at: llvm/test/Analysis/DependenceAnalysis/TripCountOverflow.ll:35
+  %add1140.i.i = add nuw nsw i32 %i, 1
+  %cmp602.i.i = icmp slt i32 %i, %n
+  br i1 %cmp602.i.i, label %for.outer, label %other
----------------
Shouldn't %n be changed to i64 (while keeping %i as i32) so this test is more representative of the condition check being removed?


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

https://reviews.llvm.org/D119261



More information about the llvm-commits mailing list