[PATCH] D22377: [SCEV] trip count calculation for loops with unknown stride

Pankaj Chawla via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 23:43:58 PDT 2016


pankajchawla added inline comments.

================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:8804
@@ +8803,3 @@
+      !PositiveStride ? APInt(BitWidth, 1, IsSigned)
+                      : IsSigned ? getSignedRange(Stride).getSignedMin()
+                                 : getUnsignedRange(Stride).getUnsignedMin();
----------------
sanjoy wrote:
> If it's all the same to you, I'd suggest avoiding a nested ternary and using an `if` instead.
I will change it to an if.


https://reviews.llvm.org/D22377





More information about the llvm-commits mailing list