[PATCH] D72436: [SCEV] get a more accurate range for AddRecExpr with nsw flag

ChenZheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 11 20:23:39 PST 2020


shchenz marked an inline comment as done.
shchenz added inline comments.


================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:5704
+                                       getSignedRangeMax(AddRec->getStart()) +
+                                           1),
+            RangeType);
----------------
nikic wrote:
> This looks a bit ugly, it might make sense to insert a newline after `getNonEmpty(` instead and only do the 4-space indent, so there's still space for the `+ 1`.
Thanks, this is formatted by clang-format. I will reformat it manually.


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

https://reviews.llvm.org/D72436





More information about the llvm-commits mailing list