PPC/s390: [builtins] Change semantics of class constructors returning primitives

PPC/s390: [builtins] Change semantics of class constructors returning primitives

Port a7c4e778466633d838c7f7d538da890853d5ea51

Original Commit Message:

    This change mirrors the semantics for derived class constructors. This
    change doesn't affect non class constructors.

    This change could potentially break web compat. More details:
    https://github.com/tc39/ecma262/pull/469

R=gsathya@chromium.org, joransiu@ca.ibm.com, bjaideep@ca.ibm.com, michael_dawson@ca.ibm.com
BUG=
LOG=N

Review-Url: https://codereview.chromium.org/2815873003
Cr-Commit-Position: refs/heads/master@{#44627}
Committed: https://chromium.googlesource.com/v8/v8/+/d9f6dd896ce92400699bb90a403894c9e5f94757

[john.yan, 2017-04-12 16:43:06]

ptal

[JaideepBajwa, 2017-04-12 16:57:57]

lgtm

[john.yan, 2017-04-12 17:00:53]

The CQ bit was checked by jyan@ca.ibm.com

[commit-bot: I haz the power, 2017-04-12 17:00:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-12 17:00:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-12 17:00:59]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-v8-committers".
Note that this has nothing to do with OWNERS files.

[john.yan, 2017-04-12 17:31:22]

The CQ bit was checked by jyan@ca.ibm.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-12 17:31:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-12 17:53:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-12 17:53:03]

Dry run: This issue passed the CQ dry run.

[JoranSiu, 2017-04-12 19:43:15]

lgtm

[john.yan, 2017-04-12 19:57:45]

The CQ bit was unchecked by jyan@ca.ibm.com

[john.yan, 2017-04-12 19:57:53]

The CQ bit was checked by jyan@ca.ibm.com

[commit-bot: I haz the power, 2017-04-12 19:57:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-12 19:59:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492027073790160, "parent_rev":
"7b300ba2e966c0d104d3755707a6d59a74944771", "commit_rev":
"d9f6dd896ce92400699bb90a403894c9e5f94757"}

[commit-bot: I haz the power, 2017-04-12 19:59:37]

Description was changed from

==========
PPC/s390: [builtins] Change semantics of class constructors returning primitives

Port a7c4e778466633d838c7f7d538da890853d5ea51

Original Commit Message:

    This change mirrors the semantics for derived class constructors. This
    change doesn't affect non class constructors.

    This change could potentially break web compat. More details:
    https://github.com/tc39/ecma262/pull/469

R=gsathya@chromium.org, joransiu@ca.ibm.com, bjaideep@ca.ibm.com,
michael_dawson@ca.ibm.com
BUG=
LOG=N
==========

to

==========
PPC/s390: [builtins] Change semantics of class constructors returning primitives

Port a7c4e778466633d838c7f7d538da890853d5ea51

Original Commit Message:

    This change mirrors the semantics for derived class constructors. This
    change doesn't affect non class constructors.

    This change could potentially break web compat. More details:
    https://github.com/tc39/ecma262/pull/469

R=gsathya@chromium.org, joransiu@ca.ibm.com, bjaideep@ca.ibm.com,
michael_dawson@ca.ibm.com
BUG=
LOG=N

Review-Url: https://codereview.chromium.org/2815873003
Cr-Commit-Position: refs/heads/master@{#44627}
Committed:
https://chromium.googlesource.com/v8/v8/+/d9f6dd896ce92400699bb90a403894c9e5f...
==========

[commit-bot: I haz the power, 2017-04-12 19:59:38]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/d9f6dd896ce92400699bb90a403894c9e5f...