https://codereview.chromium.org/622523004/diff/20001/src/ia32/full-codegen-ia32.cc
File src/ia32/full-codegen-ia32.cc (right):
https://codereview.chromium.org/622523004/diff/20001/src/ia32/full-codegen-ia32.cc#newcode2750
src/ia32/full-codegen-ia32.cc:2750: // - this (receiver) <--
LoadFromSuper will pop here and below.
LoadKeyedFromSuper in the comment
https://codereview.chromium.org/622523004/diff/20001/src/runtime/runtime-classes.cc
File src/runtime/runtime-classes.cc (right):
https://codereview.chromium.org/622523004/diff/20001/src/runtime/runtime-classes.cc#newcode98
src/runtime/runtime-classes.cc:98: if (!Runtime::ToName(isolate,
key).ToHandle(&name)) {
I think this should look like:
ASSIGN_RETURN_ON_EXCEPTION(isolate, name, Runtime::ToName(isolate,
key), Object);
to correctly handle propagate potential exception from toString() and
not replace it with UnsupportedSuper.
https://codereview.chromium.org/622523004/diff/20001/src/runtime/runtime-classes.cc#newcode102
src/runtime/runtime-classes.cc:102: if (name->AsArrayIndex(&index)) {
And this case should be handled before Runtime::ToName(). Check the
other ToName() usages.
https://codereview.chromium.org/622523004/diff/20001/src/runtime/runtime.cc
File src/runtime/runtime.cc (right):
https://codereview.chromium.org/622523004/diff/20001/src/runtime/runtime.cc#newcode466
src/runtime/runtime.cc:466: MUST_USE_RESULT
Remove MUST_USE_RESULT as it is already in the .h file.
https://codereview.chromium.org/622523004/diff/40001/test/mjsunit/harmony/super.js
File test/mjsunit/harmony/super.js (right):
https://codereview.chromium.org/622523004/diff/40001/test/mjsunit/harmony/super.js#newcode168
test/mjsunit/harmony/super.js:168: super[1]; // Indexed properties
unsupported yet.
This case seems to be checked in TestUnsupportedCases().
https://codereview.chromium.org/622523004/diff/40001/test/mjsunit/harmony/super.js#newcode177
test/mjsunit/harmony/super.js:177: derived.testGetterWithToString();
And please add a test for the toString() returning "1".
https://codereview.chromium.org/622523004/diff/40001/test/mjsunit/harmony/super.js#newcode638
test/mjsunit/harmony/super.js:638:
assertThrows(function(){f2.toMethod(o)(15);}, ReferenceError);
f1?
https://codereview.chromium.org/622523004/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
