RESOLVED FIXED 186898
ensureWritableX should only convert away from CoW when it will succeed 
https://bugs.webkit.org/show_bug.cgi?id=186898
Summary ensureWritableX should only convert away from CoW when it will succeed
Saam Barati
Reported 2018-06-21 15:09:17 PDT
Otherwise, we'll end up seeing non CoW in the array profile during OSR exit
Attachments
patch (8.22 KB, patch)
2018-06-21 16:34 PDT, Saam Barati 		keith_miller: review+
DetailsFormatted DiffDiff
patch for landing (8.38 KB, patch)
2018-06-21 16:59 PDT, Saam Barati 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Saam Barati
Comment 1 2018-06-21 16:34:18 PDT
Created attachment 343288 [details] patch
Keith Miller
Comment 2 2018-06-21 16:38:08 PDT
Comment on attachment 343288 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=343288&action=review > Source/JavaScriptCore/runtime/JSObject.cpp:1482 > + if (hasInt32(indexingMode())) { Nit I would do: leastUpperBoundOfIndexingTypes(indexingType() & IndexingShapeMask, Int32Shape) == Int32Shape > Source/JavaScriptCore/runtime/JSObject.cpp:1517 > + if (hasDouble(indexingMode()) || hasInt32(indexingMode())) { Nit I would do: leastUpperBoundOfIndexingTypes(indexingType() & IndexingShapeMask, DoubleShape) == DoubleShape > Source/JavaScriptCore/runtime/JSObject.cpp:1556 > + if (hasContiguous(indexingMode()) || hasInt32(indexingMode()) || hasDouble(indexingMode())) { Nit I would do: leastUpperBoundOfIndexingTypes(indexingType() & IndexingShapeMask, ContiguousShape) == ContiguousShape
Keith Miller
Comment 3 2018-06-21 16:38:27 PDT
r=me
Saam Barati
Comment 4 2018-06-21 16:59:36 PDT
Created attachment 343290 [details] patch for landing
WebKit Commit Bot
Comment 5 2018-06-22 14:02:07 PDT
Comment on attachment 343290 [details] patch for landing Clearing flags on attachment: 343290 Committed r233098: <https://trac.webkit.org/changeset/233098>
WebKit Commit Bot
Comment 6 2018-06-22 14:02:08 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7 2018-06-22 14:03:19 PDT
<rdar://problem/41380999>
Note You need to log in before you can comment on or make changes to this bug.