MIPS: [runtime] Throw exception for derived constructors in correct context.

MIPS: [runtime] Throw exception for derived constructors in correct context.

Port for https://codereview.chromium.org/1593553002/

Original commit message:
When derived constructors return a non-object (or not undefined) we
currently throw an exception directly in the callee context. This was
achieved by desugaring the return statement for derived classes. To
be spec compliamnt a separate ConstructStubForDerived is introduced.
Instead of trowing directly, the desugared return statement inside
a derived constructor only returns an integer to indicate an incompatible
result.

BUG=v8:4509
LOG=n

Committed: https://crrev.com/316ef286765deab3079d8dda4853da9c5dc842d4
Cr-Commit-Position: refs/heads/master@{#33340}

[balazs.kilvady, 2016-01-15 13:21:30]

balazs.kilvady@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, cbruni@chromium.org, ivica.bogosavljevic@imgtec.com,
paul.lind@imgtec.com

[Camillo Bruni, 2016-01-15 13:51:47]

https://codereview.chromium.org/1593603002/diff/1/src/mips/builtins-mips.cc
File src/mips/builtins-mips.cc (right):

https://codereview.chromium.org/1593603002/diff/1/src/mips/builtins-mips.cc#n...
src/mips/builtins-mips.cc:696: __ push(v0);
Sorry, I figured out that I don't have to pass an argument to the runtime
function, so you could drop the push here.

https://codereview.chromium.org/1593603002/diff/1/src/mips64/builtins-mips64.cc
File src/mips64/builtins-mips64.cc (right):

https://codereview.chromium.org/1593603002/diff/1/src/mips64/builtins-mips64....
src/mips64/builtins-mips64.cc:689: __ push(v0);
same here ;)

[balazs.kilvady, 2016-01-15 13:59:44]

https://codereview.chromium.org/1593603002/diff/1/src/mips/builtins-mips.cc
File src/mips/builtins-mips.cc (right):

https://codereview.chromium.org/1593603002/diff/1/src/mips/builtins-mips.cc#n...
src/mips/builtins-mips.cc:696: __ push(v0);
On 2016/01/15 13:51:47, cbruni wrote:
> Sorry, I figured out that I don't have to pass an argument to the runtime
> function, so you could drop the push here.

Done.

https://codereview.chromium.org/1593603002/diff/1/src/mips64/builtins-mips64.cc
File src/mips64/builtins-mips64.cc (right):

https://codereview.chromium.org/1593603002/diff/1/src/mips64/builtins-mips64....
src/mips64/builtins-mips64.cc:689: __ push(v0);
On 2016/01/15 13:51:47, cbruni wrote:
> same here ;)

Done.

[Camillo Bruni, 2016-01-15 15:33:27]

lgtm

[balazs.kilvady, 2016-01-15 16:37:50]

The CQ bit was checked by balazs.kilvady@imgtec.com

[commit-bot: I haz the power, 2016-01-15 16:37:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1593603002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1593603002/20001

[commit-bot: I haz the power, 2016-01-15 16:57:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-15 16:58:49]

Description was changed from

==========
MIPS: [runtime] Throw exception for derived constructors in correct context.

Port for https://codereview.chromium.org/1593553002/

Original commit message:
When derived constructors return a non-object (or not undefined) we
currently throw an exception directly in the callee context. This was
achieved by desugaring the return statement for derived classes. To
be spec compliamnt a separate ConstructStubForDerived is introduced.
Instead of trowing directly, the desugared return statement inside
a derived constructor only returns an integer to indicate an incompatible
result.

BUG=v8:4509
LOG=n
==========

to

==========
MIPS: [runtime] Throw exception for derived constructors in correct context.

Port for https://codereview.chromium.org/1593553002/

Original commit message:
When derived constructors return a non-object (or not undefined) we
currently throw an exception directly in the callee context. This was
achieved by desugaring the return statement for derived classes. To
be spec compliamnt a separate ConstructStubForDerived is introduced.
Instead of trowing directly, the desugared return statement inside
a derived constructor only returns an integer to indicate an incompatible
result.

BUG=v8:4509
LOG=n

Committed: https://crrev.com/316ef286765deab3079d8dda4853da9c5dc842d4
Cr-Commit-Position: refs/heads/master@{#33340}
==========

[commit-bot: I haz the power, 2016-01-15 16:58:50]

Patchset 2 (id:??) landed as
https://crrev.com/316ef286765deab3079d8dda4853da9c5dc842d4
Cr-Commit-Position: refs/heads/master@{#33340}