Bug 170974

Summary: Should not need to nop-pad between register allocation and a label
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: ysuzuki
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   

Filip Pizlo
Reported 2017-04-18 18:40:17 PDT
If I have code like: GPRTemporary blah GPRReg blahGPR = blah.gpr() MacroAssembler::Label = m_jit.label(); It may currently fail register allocation validation, presumably because the validation thinks that the label is inside the register allocation rather than after it. The current workaround is to insert a nop: GPRTemporary blah GPRReg blahGPR = blah.gpr() m_jit.nop(); MacroAssembler::Label = m_jit.label(); We should probably fix that eventually.
Attachments
Note You need to log in before you can comment on or make changes to this bug.