Skip to content

Conversation

@vijaysun-omr
Copy link
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
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
Contributor Author

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

Copy link
Contributor

@hzongaro hzongaro 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. Thanks!

@hzongaro hzongaro self-assigned this Jan 8, 2025
Signed-off-by: Vijay Sundaresan <vijaysun@ca.ibm.com>
@0xdaryl
Copy link
Contributor

0xdaryl commented Jan 8, 2025

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

@hzongaro hzongaro merged commit 9e718a6 into eclipse-omr:master Jan 8, 2025
1 check passed
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