On 2015/05/21 17:56:20, arv wrote:
On 2015/05/20 22:38:34, arv wrote:
> I've cleanly applied the patch now.
>
> There is some problem related to call spread.
> test/mjsunit/harmony/spread-call-super-property.js is failing now.
>
> I'm also seeing an invalid BailoutId in the same test with TF.
>
> I'll look closer tomorrow.
Caitlin, it actually seems like it is the rest params that makes this
fail.
The invalid BailoutId or the other thing?
IIRC we're doing some finicky stuff because of newTarget being pushed on the
stack, (so we change the restIndex depending on if newTarget is pushed or
not).
Did you make any changes to that code?
https://codereview.chromium.org/1135243004/
--
--
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.
