Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-12-28 11:28:38 +00:00
Code Issues Actions 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 1dd735b1d6
commit 210a8ffb30
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);
}
// 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
// Else it should stop at the start of the next successor
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
// end in order in order to maintain RPO
for (size_t i = 0; i < bodyBlocks.length(); i++) {
graph_.moveBlockToEnd(bodyBlocks[i]);
(*state.lookupswitch.bodies)[i] = bodyBlocks[i];
}
graph_.moveBlockToEnd(bodyBlocks[0]);
// Create control flow info
ControlFlowInfo switchinfo(cfgStack_.length(), exitpc);