Skip to content

GlobalDepsAnalyzer Optimizations#347

Merged
yuri91 merged 4 commits intomasterfrom
llc-optimizations
Feb 13, 2026
Merged

GlobalDepsAnalyzer Optimizations#347
yuri91 merged 4 commits intomasterfrom
llc-optimizations

Conversation

@andmadri
Copy link

@andmadri andmadri commented Feb 11, 2026 •
edited
Loading

The PR consists of:

  • Changing containers to DenseSet or DenseMap
  • Previously, there was logic executed between the two calls that justified running it again. That intermediate step has since been removed, making the second invocation redundant.
  • Removed the container for devirtualizedCalls. We are not using it at all.
  • Decied at the end to keep simplifyCalls, couldn't find a reason why we had it in the first place and not just had the logic on the main loop.

@andmadri andmadri requested a review from yuri91 February 11, 2026 08:30
Andres Madrid Ucha added 3 commits February 12, 2026 10:26
GDA: replace std to llvm containers
c339413
GDA: cleanup processEnqueuedFunction call
19cab50
GDA: cleanup devirtualizedCalls
c6623d7 
It seems like the logic for processing devirtualizedCalls
was removed on previous commits but somehow we were still
inserting pairs into the vector for no good reason.
@yuri91 yuri91 added the hydra Run hydra on this PR label Feb 12, 2026
GDA: merged two loops for opt on function list
f11bb4d 
We used to loop twice on all the functions of a module when we called
GDA on opt.
@yuri91 yuri91 merged commit 79f9a64 into master Feb 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yuri91 yuri91 yuri91 approved these changes

Assignees

No one assigned

Labels

hydra Run hydra on this PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andmadri @yuri91