wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 754155 - Fix RPO issue in LookupSwitch compilation. (r=dvander)

Browse Source
This commit is contained in:
Kannan Vijayan 2012-05-11 15:41:43 -04:00
parent 139766ee08
commit 83dbbe4042
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
js/src/ion/IonBuilder.cpp
View File

@ -1459,6 +1459,9 @@ IonBuilder::processNextLookupSwitchCase(CFGState &state)
successor->addPredecessor(current); successor->addPredecessor(current);
} }
// Move next body block to end to maintain RPO.
graph_.moveBlockToEnd(successor);
// If this is the last successor the block should stop at the end of the lookupswitch // If this is the last successor the block should stop at the end of the lookupswitch
// Else it should stop at the start of the next successor // Else it should stop at the start of the next successor
if (curBlock + 1 < state.lookupswitch.bodies->length()) if (curBlock + 1 < state.lookupswitch.bodies->length())
@ -2209,9 +2212,9 @@ IonBuilder::lookupSwitch(JSOp op, jssrcnote *sn)
// Fill bodies in CFGState using bodies in bodyBlocks, move them to // Fill bodies in CFGState using bodies in bodyBlocks, move them to
// end in order in order to maintain RPO // end in order in order to maintain RPO
for (size_t i = 0; i < bodyBlocks.length(); i++) { for (size_t i = 0; i < bodyBlocks.length(); i++) {
graph_.moveBlockToEnd(bodyBlocks[i]);
(*state.lookupswitch.bodies)[i] = bodyBlocks[i]; (*state.lookupswitch.bodies)[i] = bodyBlocks[i];
} }
graph_.moveBlockToEnd(bodyBlocks[0]);
// Create control flow info // Create control flow info
ControlFlowInfo switchinfo(cfgStack_.length(), exitpc); ControlFlowInfo switchinfo(cfgStack_.length(), exitpc);