[turbofan] Optimize loads of global constants in JSTypedLowering.

[turbofan] Optimize loads of global constants in JSTypedLowering.

R=mstarzinger@chromium.org,verwaest@chromium.org
BUG=

Committed: https://crrev.com/069e6b2f5fc5b6cb8d31197d6e0c2c86190f18fa
Cr-Commit-Position: refs/heads/master@{#27792}

[titzer, 2015-04-13 09:04:09]

On 2015/04/13 08:17:03, titzer wrote:

I should add some unit tests for JSTypedLowering...

[titzer, 2015-04-13 12:49:42]

On 2015/04/13 09:04:09, titzer wrote:
> On 2015/04/13 08:17:03, titzer wrote:
> 
> I should add some unit tests for JSTypedLowering...

Done.

[Michael Starzinger, 2015-04-13 15:04:29]

LGTM with nits.

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-inf...
File test/mjsunit/global-infinity-strict.js (right):

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-inf...
test/mjsunit/global-infinity-strict.js:4: "use strict";
nit: After the flags for consistency?

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-nan...
File test/mjsunit/global-nan-strict.js (right):

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-nan...
test/mjsunit/global-nan-strict.js:4: "use strict";
nit: After the flags for consistency?

https://codereview.chromium.org/1077343002/diff/20001/test/unittests/compiler...
File test/unittests/compiler/js-typed-lowering-unittest.cc (right):

https://codereview.chromium.org/1077343002/diff/20001/test/unittests/compiler...
test/unittests/compiler/js-typed-lowering-unittest.cc:881:
node->AppendInput(zone(), UndefinedConstant());
nit: s/UndefinedConstant/EmptyFrameState/

[titzer, 2015-04-13 15:23:27]

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-inf...
File test/mjsunit/global-infinity-strict.js (right):

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-inf...
test/mjsunit/global-infinity-strict.js:4: "use strict";
On 2015/04/13 15:04:29, Michael Starzinger wrote:
> nit: After the flags for consistency?

Done.

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-nan...
File test/mjsunit/global-nan-strict.js (right):

https://codereview.chromium.org/1077343002/diff/20001/test/mjsunit/global-nan...
test/mjsunit/global-nan-strict.js:4: "use strict";
On 2015/04/13 15:04:29, Michael Starzinger wrote:
> nit: After the flags for consistency?

Done.

https://codereview.chromium.org/1077343002/diff/20001/test/unittests/compiler...
File test/unittests/compiler/js-typed-lowering-unittest.cc (right):

https://codereview.chromium.org/1077343002/diff/20001/test/unittests/compiler...
test/unittests/compiler/js-typed-lowering-unittest.cc:881:
node->AppendInput(zone(), UndefinedConstant());
On 2015/04/13 15:04:29, Michael Starzinger wrote:
> nit: s/UndefinedConstant/EmptyFrameState/

Done.

[titzer, 2015-04-13 15:23:40]

The CQ bit was checked by titzer@chromium.org

[titzer, 2015-04-13 15:23:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1077343002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-04-13 15:23:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1077343002/40001

[commit-bot: I haz the power, 2015-04-13 16:22:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-13 16:22:23]

Patchset 3 (id:??) landed as
https://crrev.com/069e6b2f5fc5b6cb8d31197d6e0c2c86190f18fa
Cr-Commit-Position: refs/heads/master@{#27792}