Summary: | [FTL] Support ArrayPop for ArrayStorage | ||||||||
---|---|---|---|---|---|---|---|---|---|
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-02-14 03:02:39 PST
Created attachment 333778 [details]
Patch
Created attachment 333779 [details]
Patch
Comment on attachment 333779 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=333779&action=review r=me > Source/JavaScriptCore/ftl/FTLLowerDFGToB3.cpp:4768 > + m_out.aboveOrEqual(newLength, m_out.load32(storage, m_heaps.Butterfly_vectorLength)), usually(slowCase), usually(popCheckCase)); You do 'usually' twice here. You probably want 'rarely(slowCase)' Comment on attachment 333779 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=333779&action=review Thanks! >> Source/JavaScriptCore/ftl/FTLLowerDFGToB3.cpp:4768 >> + m_out.aboveOrEqual(newLength, m_out.load32(storage, m_heaps.Butterfly_vectorLength)), usually(slowCase), usually(popCheckCase)); > > You do 'usually' twice here. You probably want 'rarely(slowCase)' Right, I'll change this to `rarely(slowCase)`. Committed r228727: <https://trac.webkit.org/changeset/228727> |