Remove strong mode support from property loads.

Remove strong mode support from property loads.

R=rossberg@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
BUG=v8:3956
LOG=n

Committed: https://crrev.com/305a36e0d41f22d79b60daea70050e58f960bd8e
Cr-Commit-Position: refs/heads/master@{#34067}

[rmcilroy, 2016-02-16 15:04:53]

rmcilroy@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[rmcilroy, 2016-02-16 15:04:54]

Interpreter changes lgtm once bots are happy.

[Michael Starzinger, 2016-02-16 15:26:18]

Description was changed from

==========
Remove strong mode support from property loads.

R=
BUG=v8:3956
LOG=n
==========

to

==========
Remove strong mode support from property loads.

R=rossberg@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
BUG=v8:3956
LOG=n
==========

[Michael Starzinger, 2016-02-16 15:26:18]

mstarzinger@chromium.org changed reviewers:
+ bmeurer@chromium.org, rossberg@chromium.org, verwaest@chromium.org
- rmcilroy@chromium.org

[rmcilroy, 2016-02-16 17:08:41]

rmcilroy@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[rmcilroy, 2016-02-16 17:08:42]

Still lgtm.

https://codereview.chromium.org/1700993002/diff/80001/src/interpreter/bytecod...
File src/interpreter/bytecode-generator.cc (right):

https://codereview.chromium.org/1700993002/diff/80001/src/interpreter/bytecod...
src/interpreter/bytecode-generator.cc:1867: // TODO(oth): Abstraction not
suitable for 3 args, will over-alllcate regs.
/s/alllcate/allocate

[Michael Starzinger, 2016-02-16 17:13:17]

https://codereview.chromium.org/1700993002/diff/80001/src/interpreter/bytecod...
File src/interpreter/bytecode-generator.cc (right):

https://codereview.chromium.org/1700993002/diff/80001/src/interpreter/bytecod...
src/interpreter/bytecode-generator.cc:1867: // TODO(oth): Abstraction not
suitable for 3 args, will over-alllcate regs.
On 2016/02/16 17:08:42, rmcilroy wrote:
> /s/alllcate/allocate

Done.

[Michael Starzinger, 2016-02-16 17:14:23]

This is ready for review.

Andreas: Please sign off on the whole change.
Benedikt: PTAL at the compiler changes.
Toon: PTAL at the stub changes.

[rossberg, 2016-02-16 17:31:07]

lgtm (timbre)

[Toon Verwaest, 2016-02-17 09:00:51]

So much yes! Land it before it conflicts with someone else ;-)

LGTM

[Michael Starzinger, 2016-02-17 10:24:11]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-02-17 10:24:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
rmcilroy@chromium.org
Link to the patchset: https://codereview.chromium.org/1700993002/#ps100001
(title: "Fix comment.")

[commit-bot: I haz the power, 2016-02-17 10:24:20]

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

[commit-bot: I haz the power, 2016-02-17 10:30:19]

Description was changed from

==========
Remove strong mode support from property loads.

R=rossberg@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
BUG=v8:3956
LOG=n
==========

to

==========
Remove strong mode support from property loads.

R=rossberg@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
BUG=v8:3956
LOG=n
==========

[commit-bot: I haz the power, 2016-02-17 10:30:20]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-02-17 10:30:55]

Description was changed from

==========
Remove strong mode support from property loads.

R=rossberg@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
BUG=v8:3956
LOG=n
==========

to

==========
Remove strong mode support from property loads.

R=rossberg@chromium.org,bmeurer@chromium.org,verwaest@chromium.org
BUG=v8:3956
LOG=n

Committed: https://crrev.com/305a36e0d41f22d79b60daea70050e58f960bd8e
Cr-Commit-Position: refs/heads/master@{#34067}
==========

[commit-bot: I haz the power, 2016-02-17 10:30:56]

Patchset 6 (id:??) landed as
https://crrev.com/305a36e0d41f22d79b60daea70050e58f960bd8e
Cr-Commit-Position: refs/heads/master@{#34067}