[turbofan] No need to load the feedback vector on the JS graph level.

[turbofan] No need to load the feedback vector on the JS graph level.

Fix JSLoadGlobal/JSStoreGlobal, JSLoadNamed/JSStoreNamed and
JSLoadProperty/JSStoreProperty to take the current function closure
instead of the type feedback as input, and load the feedback vector
from the closure as required (in JSGenericLowering). There's still
JSLoadDynamic left to be done, and then we don't have any machine
operators in the initial JS graph left.

There'll be more refactoring on the JSGenericLowering in a follow-up
CL, which takes care of the current code duplication and also tries
to use the more efficient LoadIC/KeyedLoadIC instead of the current
LoadICInOptimizedCode/KeyedLoadICInOptimizedCode (and same for store
ICs) whenever possible.

R=jarin@chromium.org

Committed: https://crrev.com/9ee47bbc0964f18460fc9181140181dd6cad62ec
Cr-Commit-Position: refs/heads/master@{#33854}

[Benedikt Meurer, 2016-02-10 06:35:15]

Hey Jaro,

First step towards sane JS graph level again.
Please take a look.

Thanks,
Benedikt

[Jarin, 2016-02-10 06:58:15]

lgtm.

[Benedikt Meurer, 2016-02-10 06:58:29]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-02-10 06:58:31]

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

[commit-bot: I haz the power, 2016-02-10 07:00:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-10 07:01:01]

Description was changed from

==========
[turbofan] No need to load the feedback vector on the JS graph level.

Fix JSLoadGlobal/JSStoreGlobal, JSLoadNamed/JSStoreNamed and
JSLoadProperty/JSStoreProperty to take the current function closure
instead of the type feedback as input, and load the feedback vector
from the closure as required (in JSGenericLowering). There's still
JSLoadDynamic left to be done, and then we don't have any machine
operators in the initial JS graph left.

There'll be more refactoring on the JSGenericLowering in a follow-up
CL, which takes care of the current code duplication and also tries
to use the more efficient LoadIC/KeyedLoadIC instead of the current
LoadICInOptimizedCode/KeyedLoadICInOptimizedCode (and same for store
ICs) whenever possible.

R=jarin@chromium.org
==========

to

==========
[turbofan] No need to load the feedback vector on the JS graph level.

Fix JSLoadGlobal/JSStoreGlobal, JSLoadNamed/JSStoreNamed and
JSLoadProperty/JSStoreProperty to take the current function closure
instead of the type feedback as input, and load the feedback vector
from the closure as required (in JSGenericLowering). There's still
JSLoadDynamic left to be done, and then we don't have any machine
operators in the initial JS graph left.

There'll be more refactoring on the JSGenericLowering in a follow-up
CL, which takes care of the current code duplication and also tries
to use the more efficient LoadIC/KeyedLoadIC instead of the current
LoadICInOptimizedCode/KeyedLoadICInOptimizedCode (and same for store
ICs) whenever possible.

R=jarin@chromium.org

Committed: https://crrev.com/9ee47bbc0964f18460fc9181140181dd6cad62ec
Cr-Commit-Position: refs/heads/master@{#33854}
==========

[commit-bot: I haz the power, 2016-02-10 07:01:02]

Patchset 1 (id:??) landed as
https://crrev.com/9ee47bbc0964f18460fc9181140181dd6cad62ec
Cr-Commit-Position: refs/heads/master@{#33854}