WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 193127
171421
[JSC] Add @isUndefinedOrNullIgnoringMasquerade()
https://bugs.webkit.org/show_bug.cgi?id=171421
Summary
[JSC] Add @isUndefinedOrNullIgnoringMasquerade()
Yusuke Suzuki
Reported
2017-04-28 00:51:47 PDT
In builtin JS, we have bunch of if (object === null || object === @undefined) thing. Previously, this is written as follows. if (object == null) However, the above one recognizes document.all as null/undefined due to its masquerade. So, they are rewritten. But the above thing is, 1. From JS developer, it looks very storange because it seems `== null` is better. 2. It bloats bytecodes, effectively prevents us from inlining. Keep builtin JS function small is a bit important in terms of inlining. So, to make the meaning explicit and tighten bytecode size, we would like to introduce a new bytecode, op_is_undefined_or_null_ignoring_masquerade.
Attachments
Add attachment
proposed patch, testcase, etc.
Alexey Shvayka
Comment 1
2021-01-04 11:01:14 PST
(In reply to Yusuke Suzuki from
comment #0
)
> So, to make the meaning explicit and tighten bytecode size, we would like to > introduce a new bytecode, op_is_undefined_or_null_ignoring_masquerade.
OpIsUndefinedOrNull was introduced in
r239761
. *** This bug has been marked as a duplicate of
bug 193127
***
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