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

Deprecate splitWarmAndColdBlocksPhase #7102

Merged
merged 1 commit into from Sep 18, 2019

Conversation

fjeremic
Copy link
Contributor

As discussed in #7049 this phase no longer serves any
purpose since the support for allocating warm and cold blocks at the
code cache level has been removed from the compiler.

Closes: #7049
Signed-off-by: Filip Jeremic fjeremic@ca.ibm.com

@fjeremic
Copy link
Contributor Author

We can proceed with the review, but before we merge this I will perform several performance runs to ensure removal of this phase has no measurable effect on throughput. As such I'll mark this as WIP until the performance numbers are posted.

@fjeremic fjeremic changed the title Deprecate splitWarmAndColdBlocksPhase WIP: Deprecate splitWarmAndColdBlocksPhase Sep 16, 2019
As discussed in eclipse-openj9#7049 this phase no longer serves any
purpose since the support for allocating warm and cold blocks at the
code cache level has been removed from the compiler.

Closes: eclipse-openj9#7049
Signed-off-by: Filip Jeremic <fjeremic@ca.ibm.com>
@fjeremic
Copy link
Contributor Author

Finished the performance runs. On DayTrader7 the delta with and without the change is well within the noise range:

----------------------------------------------------------------------------------
Run Name: J8R5-2019-09-17-daytrader7-baseline
----------------------------------------------------------------------------------
Run ID 1 Score: 10238.125
Run ID 2 Score: 10308.9852558543
Run ID 3 Score: 10343.4410480349
Run ID 4 Score: 10311.9458987784
Run ID 5 Score: 10174.5338208409
Run ID 6 Score: 10311.5693112467
Run ID 7 Score: 10283.28125
Run ID 8 Score: 10357.6838235294
Run ID 9 Score: 10217.2038327526
Run ID 10 Score: 10261.0386029412

Average: 10280.7807843978

----------------------------------------------------------------------------------
Run Name: J8R5-2019-09-17-daytrader7-deprecate-split
----------------------------------------------------------------------------------
Run ID 1 Score: 10166.7155172414
Run ID 2 Score: 10283.8362068966
Run ID 3 Score: 10276.79163034
Run ID 4 Score: 10381.719038817
Run ID 5 Score: 10342.7163904236
Run ID 6 Score: 10324.2993907746
Run ID 7 Score: 10342.34375
Run ID 8 Score: 10087.4434782609
Run ID 9 Score: 10239.8160073597
Run ID 10 Score: 10222.4242424242

Average: 10266.8105652538

The data shows that deprecating this phase has no effect on performance.

@fjeremic fjeremic changed the title WIP: Deprecate splitWarmAndColdBlocksPhase Deprecate splitWarmAndColdBlocksPhase Sep 18, 2019
@fjeremic
Copy link
Contributor Author

Jenkins test sanity zlinux jdk8

Copy link
Contributor

@0xdaryl 0xdaryl left a comment

Choose a reason for hiding this comment

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

Yay!

@0xdaryl 0xdaryl self-assigned this Sep 18, 2019
@0xdaryl 0xdaryl merged commit 12ef37b into eclipse-openj9:master Sep 18, 2019
@fjeremic fjeremic deleted the deprecate-split branch September 19, 2019 01:03
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.

Why does Z use splitWarmAndColdBlocks and other codegens do not?
2 participants