[PATCH] D46178: [SCEV] Reduce the number of invocation to non trivial getExact function

[Javed Absar via Phabricator via llvm-commits]

javed.absar added a comment.

AFAIU the correctness check would now get  gated on STATS colelction and that may not be right. I would recommend separating the two.



================
Comment at: lib/Analysis/ScalarEvolution.cpp:6506
+#if LLVM_ENABLE_STATS
+  auto *BEExact = Result.getExact(L, this);
+  if (BEExact != getCouldNotCompute()) {
----------------
Please s/auto/"const SCEV *" to be consistent with other such instances in SCEV.



https://reviews.llvm.org/D46178