Permalink
Show file tree
Hide file tree
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Avoid allocating useless landingBlocks in DFGByteCodeParser::handleIn…
…lining() https://bugs.webkit.org/show_bug.cgi?id=177926 Reviewed by Saam Barati. When doing polyvariant inlining, there used to be a landing block for each callee, each of which was then linked to a continuation block. With this change, we allocate the continuation block first, and pass it to the inlining routine so that op_ret in the callee link directly to it. The only subtlety is that when inlining an intrinsic we must do the jump by hand, and also remember to call processSetLocalQueue with nextOffset before it. * dfg/DFGByteCodeParser.cpp: (JSC::DFG::ByteCodeParser::inlineCall): (JSC::DFG::ByteCodeParser::attemptToInlineCall): (JSC::DFG::ByteCodeParser::handleInlining): (JSC::DFG::ByteCodeParser::InlineStackEntry::InlineStackEntry): (JSC::DFG::ByteCodeParser::parse): Canonical link: https://commits.webkit.org/194401@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@223159 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- Loading branch information
Robin Morisset
committed
Oct 11, 2017
1 parent
ef69b4a
commit d3a05f6e0cfcc8563250ebb2481d1cccf1082058
Showing
2 changed files
with
58 additions
and
60 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters