[llvm] [LoopInterchange] Bail out for Scalar Dependencies (PR #119345)
Michael Kruse via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 7 06:06:59 PST 2025
Meinersbur wrote:
I honestly never understood why a dependency that is "scalar" (that is, the dimension is not referenced in the index expression) is handled fundamentally different. If not references it should be equivalent to accessing (any of) the previous iteration. How about we remove
```
if (D->isScalar(II)) {
Direction = 'S';
Dep.push_back(Direction);
```
in populateDependencyMatrix entirely, so a dependency is never `S`?
In any case, the patch in the current form should be conservatively correct.
https://github.com/llvm/llvm-project/pull/119345
More information about the llvm-commits
mailing list