[llvm] [LAA] Relax HasSameSize check on couldPrevertStoreLoadForward (PR #159537)
David Sherwood via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 3 01:34:05 PDT 2026
================
@@ -2211,8 +2211,12 @@ MemoryDepChecker::getDependenceDistanceStrideAndSize(
return Dependence::Unknown;
}
+ // TODO: Remove this.
+ bool HasSameSize = AStoreSz == BStoreSz;
----------------
david-arm wrote:
The name of this and the parameter to the function seems ambiguous because there are two sizes in play here:
1. The type store size
2. The type alloc size
Can you make this more specific, i.e. something like `HasSameStoreSize`?
https://github.com/llvm/llvm-project/pull/159537
More information about the llvm-commits
mailing list