Skip to content

LLVM: add AggregateAllocaSplitting transformation#1649

Merged
phate merged 7 commits into
masterfrom
RvsdgTreePrinter-SplitableAllocas
May 17, 2026
Merged

LLVM: add AggregateAllocaSplitting transformation#1649
phate merged 7 commits into
masterfrom
RvsdgTreePrinter-SplitableAllocas

Conversation

@phate
Copy link
Copy Markdown
Owner

@phate phate commented May 10, 2026

No description provided.

@phate phate changed the title Rvsdg tree printer splitable allocas LLVM: add AggregateAllocaSplitting transformation May 10, 2026
@phate phate force-pushed the RvsdgTreePrinter-SplitableAllocas branch 3 times, most recently from e43c5b2 to 072fc75 Compare May 12, 2026 04:24
@phate phate marked this pull request as ready for review May 13, 2026 05:37
@phate phate requested a review from haved May 13, 2026 05:37
@phate phate force-pushed the RvsdgTreePrinter-SplitableAllocas branch 2 times, most recently from 5a6940f to 9187240 Compare May 16, 2026 10:49
@phate phate force-pushed the RvsdgTreePrinter-SplitableAllocas branch from 9870b70 to e049378 Compare May 16, 2026 11:19
@phate phate enabled auto-merge (squash) May 16, 2026 11:19
@phate
Copy link
Copy Markdown
Owner Author

phate commented May 16, 2026

@haved This one is now ready for review.

haved
haved previously approved these changes May 16, 2026
Copy link
Copy Markdown
Collaborator

@haved haved left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I have some comments you can look at it you'd like. The code for finding all users of the alloca and all users of the GEP is very similar, so it makes me wish for a "reverse tracer" in the future

Comment thread jlm/llvm/opt/AggregateAllocaSplitting.hpp
Comment thread jlm/llvm/opt/AggregateAllocaSplitting.cpp Outdated
Comment thread jlm/llvm/opt/AggregateAllocaSplitting.cpp Outdated
Comment thread jlm/llvm/opt/AggregateAllocaSplitting.cpp
Comment thread jlm/llvm/opt/AggregateAllocaSplitting.cpp Outdated
Comment thread jlm/llvm/opt/AggregateAllocaSplitting.cpp
@phate
Copy link
Copy Markdown
Owner Author

phate commented May 16, 2026

The code for finding all users of the alloca and all users of the GEP is very similar, so it makes me wish for a "reverse > tracer" in the future

Yes, it is on my TODO list, but I was too lazy to factor it out for this PR now.

@phate
Copy link
Copy Markdown
Owner Author

phate commented May 16, 2026

@haved I replied to all your comments.

@phate phate merged commit 8e3f487 into master May 17, 2026
28 checks passed
@phate phate deleted the RvsdgTreePrinter-SplitableAllocas branch May 17, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants