Skip to content

Tune more inlining parameters without changing the source - #7601

Merged
hzongaro merged 1 commit into
eclipse-omr:masterfrom
vijaysun-omr:inliner-diags1
Jan 8, 2025
Merged

Tune more inlining parameters without changing the source#7601
hzongaro merged 1 commit into
eclipse-omr:masterfrom
vijaysun-omr:inliner-diags1

Conversation

@vijaysun-omr

Copy link
Copy Markdown
Contributor

Several inlining parameters can be tuned via command line options or env vars. Recently, I needed to tune some inlining parameters that could not be controlled without changing the source. This PR fixes the cases that I observed by adding env vars.

@vijaysun-omr

Copy link
Copy Markdown
Contributor Author

Jenkins build all

@vijaysun-omr vijaysun-omr changed the title WIP : Tune more inlining parameters without changing the source Tune more inlining parameters without changing the source Jan 8, 2025
@vijaysun-omr

Copy link
Copy Markdown
Contributor Author

@0xdaryl and @hzongaro fyi in case you wanted to review

@hzongaro hzongaro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Thanks!

@hzongaro hzongaro self-assigned this Jan 8, 2025
Comment thread compiler/optimizer/Inliner.cpp Outdated
Signed-off-by: Vijay Sundaresan <vijaysun@ca.ibm.com>
@0xdaryl

0xdaryl commented Jan 8, 2025

Copy link
Copy Markdown
Contributor

No need to re-run CI with the latest change.

@hzongaro
hzongaro merged commit 9e718a6 into eclipse-omr:master Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants