[turbofan] Fix various bailout points for AstGraphBuilder.

[turbofan] Fix various bailout points for AstGraphBuilder.

This introduces a bunch of new tests that test various aspects of
accessor inlining in TurboFan (without the actual inlining), and does
the appropriate fixes to the AstGraphBuilder. The actual inlining CL
will land separately (so we don't need to revert the tests and fixes
if the accessor CL has to be reverted).

R=jarin@chromium.org

Committed: https://crrev.com/986b04a62a5e3617f7c951ff91b83bcd1daa6f85
Cr-Commit-Position: refs/heads/master@{#38191}

[Benedikt Meurer, 2016-08-01 05:51:08]

The CQ bit was checked by bmeurer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-01 05:51:21]

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

[Jarin, 2016-08-01 05:55:26]

lgtm with a small request.

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
File test/mjsunit/compiler/accessor-exceptions1.js (right):

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
test/mjsunit/compiler/accessor-exceptions1.js:13: try { o.x; } catch (e) { }
Could you also check that we catch the right exception?

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
File test/mjsunit/compiler/accessor-exceptions2.js (right):

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
test/mjsunit/compiler/accessor-exceptions2.js:13: try { o.x = 2; } catch (e) { }
Could you check the exception here, please?

[Benedikt Meurer, 2016-08-01 05:55:57]

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
File test/mjsunit/compiler/accessor-exceptions1.js (right):

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
test/mjsunit/compiler/accessor-exceptions1.js:13: try { o.x; } catch (e) { }
On 2016/08/01 05:55:26, Jarin wrote:
> Could you also check that we catch the right exception?

Done.

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
File test/mjsunit/compiler/accessor-exceptions2.js (right):

https://codereview.chromium.org/2197913002/diff/1/test/mjsunit/compiler/acces...
test/mjsunit/compiler/accessor-exceptions2.js:13: try { o.x = 2; } catch (e) { }
On 2016/08/01 05:55:26, Jarin wrote:
> Could you check the exception here, please?

Done.

[Benedikt Meurer, 2016-08-01 05:56:02]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-08-01 05:56:02]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2197913002/#ps20001
(title: "Address comments")

[commit-bot: I haz the power, 2016-08-01 05:56:08]

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, 2016-08-01 06:17:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-01 06:18:55]

Description was changed from

==========
[turbofan] Fix various bailout points for AstGraphBuilder.

This introduces a bunch of new tests that test various aspects of
accessor inlining in TurboFan (without the actual inlining), and does
the appropriate fixes to the AstGraphBuilder. The actual inlining CL
will land separately (so we don't need to revert the tests and fixes
if the accessor CL has to be reverted).

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

to

==========
[turbofan] Fix various bailout points for AstGraphBuilder.

This introduces a bunch of new tests that test various aspects of
accessor inlining in TurboFan (without the actual inlining), and does
the appropriate fixes to the AstGraphBuilder. The actual inlining CL
will land separately (so we don't need to revert the tests and fixes
if the accessor CL has to be reverted).

R=jarin@chromium.org

Committed: https://crrev.com/986b04a62a5e3617f7c951ff91b83bcd1daa6f85
Cr-Commit-Position: refs/heads/master@{#38191}
==========

[commit-bot: I haz the power, 2016-08-01 06:18:56]

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