[llvm] [LoopInterchange] Bail out for Scalar Dependencies (PR #119345)

Sjoerd Meijer via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 8 05:57:28 PST 2025


sjoerdmeijer wrote:

> @sjoerdmeijer What kind of interchange do you want to preserve with `S`? IMHO the current code doesn't even understand what it should mean (or it was forgotten about at some point), so if we want to re-add it, it should get another full review.
> 
> Besides, this PR currently will just reject everything with `S`, but `DependenceInfo` may be able to determine the direction already.

Okay, excellent points. I will prepare a new revision that removes `S`.

https://github.com/llvm/llvm-project/pull/119345


More information about the llvm-commits mailing list