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.

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

Allow more expensive hot compilations under -Xtune:throughput #14563

Merged
merged 1 commit into from
Feb 22, 2022

Conversation

mpirvu
Copy link
Contributor

@mpirvu mpirvu commented Feb 18, 2022

This commit enables two changes for hot/scorching compilations
performed under the -Xtune:throughput mode:

  1. -Xjit:acceptHugeMethod is enabled, allowing more complex methods
    to be successfuly compiled. This change can be disabled by setting
    the following environment variable: TR_DontAcceptHugeMethods=1

  2. The JIT scratch memory limit is increased from 256 MB to 512 MB
    allowing compilations that need more memory to succeed. The new
    scratch memory limit value for hot compilations can be controlled
    with -Xjit:scratchSpaceLimitKBForHotCompilations=

Signed-off-by: Marius Pirvu mpirvu@ca.ibm.com

This commit enables two changes for hot/scorching compilations
performed under the -Xtune:throughput mode:

1. -Xjit:acceptHugeMethod is enabled, allowing more complex methods
to be successfuly compiled. This change can be disabled by setting
the following environment variable: TR_DontAcceptHugeMethods=1

2. The JIT scratch memory limit is increased from 256 MB to 512 MB
allowing compilations that need more memory to succeed. The new
scratch memory limit value for hot compilations can be controlled
with -Xjit:scratchSpaceLimitKBForHotCompilations=<NNN>

Signed-off-by: Marius Pirvu <mpirvu@ca.ibm.com>
@mpirvu
Copy link
Contributor Author

mpirvu commented Feb 18, 2022

FYI: @vijaysun-omr

@vijaysun-omr
Copy link
Contributor

Jenkins test extended all jdk11

@vijaysun-omr
Copy link
Contributor

Jenkins test sanity all jdk11

@vijaysun-omr
Copy link
Contributor

Jenkins test extended all jdk11

@vijaysun-omr
Copy link
Contributor

Jenkins test extended alinux64 jdk11

@vijaysun-omr
Copy link
Contributor

The AArch64 jobs seem to have hit some infra issue. I am going to merge this since this affects -Xtune:throughput only in a heuristic way and we do not have tests running yet in that mode (planned for the future), and the extended testing has passed on all other platforms (this is a cross platform change).

@vijaysun-omr vijaysun-omr merged commit e21d943 into eclipse-openj9:master Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants