[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