[llvm] LAA: thoroughly clarify stride-versioning code (NFC) (PR #97075)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 21 11:43:28 PDT 2024


================
@@ -2955,40 +2956,35 @@ void LoopAccessInfo::collectStridedAccess(Value *MemAccess) {
   // of various possible stride specializations, considering the alternatives
   // of using gather/scatters (if available).
 
-  const SCEV *MaxBTC = PSE->getSymbolicMaxBackedgeTakenCount();
+  const SCEV *BTC = PSE->getSymbolicMaxBackedgeTakenCount();
----------------
artagnon wrote:

Small caveat: I think this should be `getBackedgeTakenCount`, the exact version, and not the symbolic-max version. However, it's not going to be easy to cook up a test to support this change, but I can think about it in a follow-up.

https://github.com/llvm/llvm-project/pull/97075


More information about the llvm-commits mailing list