-
Notifications
You must be signed in to change notification settings - Fork 707
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
Split warm and cold blocks #19272
Split warm and cold blocks #19272
Conversation
gita-omr
commented
Apr 4, 2024
•
edited
edited
- identify last warm instruction during instruction selection
- the code is only enabled if -Xjit:splitWarmAndColdBlocks option is on
62d10e7
to
46ecafe
Compare
Addressed comments in the OMR PR. |
Jenkins test sanity all jdk17 |
@@ -2166,6 +2170,25 @@ J9::CodeGenerator::doInstructionSelection() | |||
if (doEvaluation) | |||
self()->evaluate(node); | |||
|
|||
if (opCode == TR::BBEnd) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this code need to be guarded with self()->comp()->getOption(TR_SplitWarmAndColdBlocks)
too? It seems only useful for that optimization.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isLastWarmBlock()
is only true if the option is on. But I added the check to make the code more clear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor change requested.
- identify last warm instruction during instruction selection - the code is only enabled if -Xjit:splitWarmAndColdBlocks option is on
46ecafe
to
17bed20
Compare
Addressed comments above. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Jenkins test sanity all jdk17 |
Jenkins test sanity xlinux jdk17 |
AIX build timed out. Linux build is waiting on an xlinux compile machine which are all offline. |