[llvm] [LAA] Fix incorrect dependency classification. (PR #70819)
Alexandros Lamprineas via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 7 07:16:46 PST 2023
================
@@ -1885,17 +1885,9 @@ MemoryDepChecker::isDependent(const MemAccessInfo &A, unsigned AIdx,
// If the induction step is negative we have to invert source and sink of the
----------------
labrinea wrote:
I intentionally left the comment since we are still inverting the src/sink parameters to getMinusSCEV(). Do you want me to rephrase it perhaps? The print order is the program order which the assertion a few lines above checks:
`assert (AIdx < BIdx && "Must pass arguments in program order");`
https://github.com/llvm/llvm-project/pull/70819
More information about the llvm-commits
mailing list