[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