Summary: | [Baseline] Remove a hack for DCE removal of NewFunction | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yusuke Suzuki <ysuzuki> | ||||||
Component: | New Bugs | Assignee: | Yusuke Suzuki <ysuzuki> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ews-watchlist, keith_miller, mark.lam, msaboff, saam, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Yusuke Suzuki
2018-05-24 06:49:15 PDT
Created attachment 341189 [details]
Patch
Created attachment 341244 [details]
Patch
Comment on attachment 341244 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=341244&action=review > Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp:6338 > + addToGraph(Phantom, scope); Put this after the set? > Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp:6375 > + set(VirtualRegister(currentInstruction[1].u.operand), addToGraph(op, OpInfo(frozen), scope)); ditto Comment on attachment 341244 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=341244&action=review Thank you! >> Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp:6338 >> + addToGraph(Phantom, scope); > > Put this after the set? Fixed. >> Source/JavaScriptCore/dfg/DFGByteCodeParser.cpp:6375 >> + set(VirtualRegister(currentInstruction[1].u.operand), addToGraph(op, OpInfo(frozen), scope)); > > ditto Fixed. Committed r232182: <https://trac.webkit.org/changeset/232182> |