[llvm] [LAA] Fix incorrect dependency classification. (PR #70819)
Alexandros Lamprineas via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 27 02:22:23 PST 2023
labrinea wrote:
> IIUC we should still swap source and sink for printing, as otherwise it may be confusing that the difference is not what we expect (Sink - Src)?
>
> Would be good to also add a test for the debug output.
I must have missed this comment. I think we should not be altering the order of Sink and Source when printing them, because it would be as if we claimed that the Sink is the Source and the Source is the Sink, which is not the case. All we are doing is inverting the subtraction operands because one preceeds the other in the memory layout. Having said that @fhahn are you happy with the updated description and rebase?
https://github.com/llvm/llvm-project/pull/70819
More information about the llvm-commits
mailing list