Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid calling getEnclosingBlock #16455

Merged
merged 1 commit into from
Dec 12, 2022
Merged

Avoid calling getEnclosingBlock #16455

merged 1 commit into from
Dec 12, 2022

Commits on Dec 12, 2022

  1. Avoid calling getEnclosingBlock 

    While performing analysis for flush elimination,
LocalFlushElimination::examineNode called TR::TreeTop::getEnclosingBlock
on the current TR::TreeTop.  As that method walks through the linked
list chaining together the TR::TreeTops to find the corresponding
BBStart, calling it in nested loops becomes very expensive when large
blocks are encountered.

Fixed this by passing the current block as an argument to
LocalFlushElimination::examineNode

Signed-off-by:  Henry Zongaro <[email protected]>
    @hzongaro
    hzongaro committed Dec 12, 2022
    Configuration menu
    Copy the full SHA
    42b0411 View commit details
    Browse the repository at this point in the history