Emit JSOp::DebugCheckSelfHosted in more places
Categories
(Core :: JavaScript Engine, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox105 | --- | fixed |
People
(Reporter: arai, Assigned: arai)
References
(Blocks 1 open bug)
Details
Attachments
(11 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details |
Bug 1609432 will add callContentFunction
and constructContentFunction
to all callsites where the callee can be content functions.
That means, in all other cases, the callee must be self-hosted function or built-in.
We can add JSOp::DebugCheckSelfHosted
to all calls except for the above 2, so that
onNativeCall
and eager evaluation keeps working.
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
Array.from
on an iterable object internally calls
%ArrayIteratorPrototype%.next
, which is effectful.
The later patches will add more coverage inside for-of loop inside
self-hosted JS, and that will expose the call on obj[Symbol.iterator]().next
,
which can be %ArrayIteratorPrototype%.next
or any other function.
Assignee | ||
Comment 2•2 years ago
|
||
Depends on D153153
Assignee | ||
Comment 3•2 years ago
|
||
Depends on D153154
Assignee | ||
Comment 4•2 years ago
|
||
Depends on D153155
Assignee | ||
Comment 5•2 years ago
|
||
Depends on D153156
Assignee | ||
Comment 6•2 years ago
|
||
Depends on D153157
Assignee | ||
Comment 7•2 years ago
|
||
Depends on D153158
Assignee | ||
Comment 8•2 years ago
|
||
Depends on D153159
Assignee | ||
Comment 9•2 years ago
|
||
Depends on D153160
Assignee | ||
Comment 10•2 years ago
|
||
Depends on D153161
Assignee | ||
Comment 11•2 years ago
|
||
Depends on D153162
Comment 12•2 years ago
|
||
Comment 13•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5a547dbb387b
https://hg.mozilla.org/mozilla-central/rev/6dec44311fb1
https://hg.mozilla.org/mozilla-central/rev/a7621b225cb3
https://hg.mozilla.org/mozilla-central/rev/52431c4b1588
https://hg.mozilla.org/mozilla-central/rev/f0554d74eedb
https://hg.mozilla.org/mozilla-central/rev/6f9dde36dbd8
https://hg.mozilla.org/mozilla-central/rev/aa5f3bf898bc
https://hg.mozilla.org/mozilla-central/rev/178d1de9debb
https://hg.mozilla.org/mozilla-central/rev/7e49b9f5abc1
https://hg.mozilla.org/mozilla-central/rev/c2480e13b36d
https://hg.mozilla.org/mozilla-central/rev/19cb3c811aae
Description
•