WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
195022
[JSC] Make Intl fields lazily-allocated
https://bugs.webkit.org/show_bug.cgi?id=195022
Summary
[JSC] Make Intl fields lazily-allocated
Yusuke Suzuki
Reported
2019-02-25 16:05:11 PST
[JSC] Make Intl fields lazily-allocated
Attachments
Patch
(25.22 KB, patch)
2019-02-25 16:08 PST
,
Yusuke Suzuki
no flags
Details
Formatted Diff
Diff
Patch
(23.70 KB, patch)
2019-02-25 16:12 PST
,
Yusuke Suzuki
mark.lam
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1
2019-02-25 16:08:55 PST
Created
attachment 362938
[details]
Patch
Yusuke Suzuki
Comment 2
2019-02-25 16:12:08 PST
Created
attachment 362941
[details]
Patch
Mark Lam
Comment 3
2019-02-25 16:47:10 PST
Comment on
attachment 362941
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=362941&action=review
r=me
> Source/JavaScriptCore/ChangeLog:10 > + 1. Make IntlObject fields including Intl.Collator lazily-alllocated because we already removed direct references from JS builtins to these constructors (@Collator etc.).
/alllocated/allocated/.
Yusuke Suzuki
Comment 4
2019-02-25 17:05:35 PST
Comment on
attachment 362941
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=362941&action=review
>> Source/JavaScriptCore/ChangeLog:10 >> + 1. Make IntlObject fields including Intl.Collator lazily-alllocated because we already removed direct references from JS builtins to these constructors (@Collator etc.). > > /alllocated/allocated/.
Oops, thanks.
Yusuke Suzuki
Comment 5
2019-02-25 18:13:25 PST
Committed
r242064
: <
https://trac.webkit.org/changeset/242064
>
Radar WebKit Bug Importer
Comment 6
2019-02-25 18:33:58 PST
<
rdar://problem/48387136
>
Yusuke Suzuki
Comment 7
2020-04-15 10:55:53 PDT
***
Bug 193549
has been marked as a duplicate of this bug. ***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug