[llvm] [LoopInterchange] Remove 'S' Scalar Dependencies (PR #119345)
Ryotaro Kasuga via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 19 23:24:44 PST 2025
================
@@ -18,7 +18,7 @@
; CHECK: --- !Missed
; CHECK-NEXT: Pass: loop-interchange
-; CHECK-NEXT: Name: UnsupportedPHI
+; CHECK-NEXT: Name: Dependence
----------------
kasuga-fj wrote:
I see, thanks. For now, I think there are two perspectives:
- This loop should be rejected by LoopInterchange for some reason.
- After scalar dependencies are handled correctly, this one should be rejected by `currentLimitations`.
IMHO, to guarantee the first point, it's better to take your current change and add some comments to describe the context of the second point, rather than XFAILS.
https://github.com/llvm/llvm-project/pull/119345
More information about the llvm-commits
mailing list