Patch forthcoming.
Created attachment 174875 [details] the patch
Comment on attachment 174875 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=174875&action=review > Source/JavaScriptCore/bytecode/CodeBlock.cpp:2202 > + JSCell* origin = m_dfgData->transitions[i].m_codeOrigin.get(); > + JSCell* from = m_dfgData->transitions[i].m_from.get(); > + JSCell* to = m_dfgData->transitions[i].m_to.get(); You could use a local reference to m_dfgData->transitions[i] to avoid getting it thrice.
(In reply to comment #2) > (From update of attachment 174875 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=174875&action=review > > > Source/JavaScriptCore/bytecode/CodeBlock.cpp:2202 > > + JSCell* origin = m_dfgData->transitions[i].m_codeOrigin.get(); > > + JSCell* from = m_dfgData->transitions[i].m_from.get(); > > + JSCell* to = m_dfgData->transitions[i].m_to.get(); > > You could use a local reference to m_dfgData->transitions[i] to avoid getting it thrice. Good point, that would be clearer. THanks!
Comment on attachment 174875 [details] the patch Attachment 174875 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14907014
Landed in http://trac.webkit.org/changeset/135097
Windows build fix landed in http://trac.webkit.org/changeset/135099