[llvm] [DA] do not handle array accesses of different offsets (PR #123436)
Sjoerd Meijer via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 25 05:06:13 PDT 2025
sjoerdmeijer wrote:
This patch has grown into doing at least 3 things, I think:
- it adds the concepts of "assumptions" to DA,
- it fixes offset handling in DA,
- and it fixes up interchange, and unroll-and jam.
I guess it would be easier to untangle this and create separate (stacked) patches for this, so that it's easier to review and maintain (in case we would need to partially revert/recommit some of this later). E.g., it looks like the assumptions addition can be tested in isolation as I see support for it being added to dump functions, i.e. we don't need the offset fix for that.
Does this make sense?
https://github.com/llvm/llvm-project/pull/123436
More information about the llvm-commits
mailing list