[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