RESOLVED FIXED 179550
[JSC] Allow poly proto for intrinsic getters 
https://bugs.webkit.org/show_bug.cgi?id=179550
Summary [JSC] Allow poly proto for intrinsic getters
Yusuke Suzuki
Reported 2017-11-10 15:04:22 PST
[JSC] Allow poly proto for intrinsic getters
Attachments
Patch (9.98 KB, patch)
2017-11-10 15:07 PST, Yusuke Suzuki 		saam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2017-11-10 15:07:39 PST
Created attachment 326637 [details] Patch
Darin Adler
Comment 2 2017-11-20 07:53:14 PST
Comment on attachment 326637 [details] Patch I’d love to review, but I’m not quite expert enough in the JIT.
Saam Barati
Comment 3 2017-11-20 08:52:08 PST
Comment on attachment 326637 [details] Patch Another interesting test would be to do this with a second object: polyProtoObject.__proto__ = null assert(target(polyProtoObject) === undefined)
Yusuke Suzuki
Comment 4 2017-11-21 02:00:14 PST
(In reply to Saam Barati from comment #3) > Comment on attachment 326637 [details] > Patch > > Another interesting test would be to do this with a second object: > polyProtoObject.__proto__ = null > assert(target(polyProtoObject) === undefined) Sounds good. Added.
Yusuke Suzuki
Comment 5 2017-11-21 02:07:59 PST
Committed r225071: <https://trac.webkit.org/changeset/225071>
Radar WebKit Bug Importer
Comment 6 2017-11-21 02:08:19 PST
<rdar://problem/35654956>
Note You need to log in before you can comment on or make changes to this bug.