[PATCH] D109821: [ScalarEvolution] Infer loop max trip count from array accesses

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 15 09:49:07 PDT 2021


jdoerfert added a comment.

drive by: this should use `llvm::getObjectSize` rather than looking for allocainst. That will allow some globals and malloc right away.



================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:7264
+  return (InferTripCount == TypeMax) ? 0 : InferTripCount;
+}
+
----------------
Style: Some more levels could be early exists to lower indention by 1-3 in the main part.


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

https://reviews.llvm.org/D109821



More information about the llvm-commits mailing list