-
Notifications
You must be signed in to change notification settings - Fork 394
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
Perform better spill placement for GPRs only on x86 #6598
Conversation
Signed-off-by: BradleyWood <bradley.wood@ibm.com>
Jenkins build xlinux,x32linux |
Can you create an issue to track implementing BSP for non-GPR register kinds? The optimization is sound for non-GPR kinds even if the current implementation needs some work in order to support it. |
@@ -1209,6 +1209,10 @@ void OMR::X86::CodeGenerator::saveBetterSpillPlacements(TR::Instruction * branch | |||
|
|||
void OMR::X86::CodeGenerator::removeBetterSpillPlacementCandidate(TR::RealRegister * realReg) | |||
{ | |||
// this mechanism only supports GPR's |
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.
Can you drop a sentence in here explaining why so it is clear to others coming across this code? (presumably interference with the register mask)
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.
See force-push
@@ -1250,6 +1254,11 @@ OMR::X86::CodeGenerator::findBetterSpillPlacement( | |||
{ | |||
TR::Instruction * placement; | |||
TR_BetterSpillPlacement * info; | |||
|
|||
// this mechanism only supports GPR's |
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.
Same comment as above for providing an explanation.
Signed-off-by: BradleyWood <bradley.wood@ibm.com>
86ebbed
to
755c5e3
Compare
Jenkins build xlinux,win,osx,x32linux |
Jenkins build xlinux,x32linux
See #6602