Bug 124086

Summary:

FTL should support AllocatePropertyStorage

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

barraclough, ggaren, mark.lam, mhahnenberg, msaboff, nrotem, oliver, sam

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

112840

Attachments:

Description	Flags
the patch	oliver: review+

Description Filip Pizlo 2013-11-08 16:45:50 PST

Patch forthcoming.

Comment 1 Filip Pizlo 2013-11-08 17:20:15 PST

Created attachment 216467 [details]
the patch

Comment 2 Oliver Hunt 2013-11-08 17:33:10 PST

Comment on attachment 216467 [details]
the patch

View in context: https://bugs.webkit.org/attachment.cgi?id=216467&action=review

r=me

> Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:4513
> +    m_jit.addPtr(JITCompiler::TrustedImm32(sizeof(IndexingHeader)), scratchGPR);

This seems like a good change

Comment 3 Filip Pizlo 2013-11-08 17:43:27 PST

Landed in http://trac.webkit.org/changeset/158983