Summary: | We should ensure that operationStrCat2 and operationStrCat3 are never passed Symbols as arguments. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Lam <mark.lam> | ||||
Component: | JavaScriptCore | Assignee: | Mark Lam <mark.lam> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | fpizlo, jfbastien, keith_miller, msaboff, rmorisset, saam, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Mark Lam
2017-11-10 16:46:02 PST
Created attachment 326667 [details]
proposed patch.
Comment on attachment 326667 [details] proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=326667&action=review r=me > Source/JavaScriptCore/dfg/DFGFixupPhase.cpp:205 > + m_insertionSet.insertNode( > + m_indexInBlock, SpecNone, Check, node->origin, > + Edge(edge.node(), NotSymbolUse)); Might be worth a comment saying why we do this. (In reply to Saam Barati from comment #2) > > Source/JavaScriptCore/dfg/DFGFixupPhase.cpp:205 > > + m_insertionSet.insertNode( > > + m_indexInBlock, SpecNone, Check, node->origin, > > + Edge(edge.node(), NotSymbolUse)); > > Might be worth a comment saying why we do this. Thanks. I will add the comment. Landed in r224735: <http://trac.webkit.org/r224735>. |