Cleanup ast numbering for super.prop in arrows

Cleanup ast numbering for super.prop in arrows

Follow up to https://codereview.chromium.org/1135243004

For now we need to call set_base_id in AstNumberingVisitor because
TurboFan needs a bailout id for super.prop in a lot of cases.

BUG=None
LOG=N
R=adamk@chromium.org, mstarzinger@chromium.org

Committed: https://crrev.com/ab0577b29c3d7cf0c451a3ecb7f85e87d8d8d160
Cr-Commit-Position: refs/heads/master@{#28689}

[arv (Not doing code reviews), 2015-05-26 20:57:23]

PTAL

I accidentally left some incorrect code in there.

[adamk, 2015-05-26 21:00:10]

https://codereview.chromium.org/1153053005/diff/1/src/compiler/ast-graph-buil...
File src/compiler/ast-graph-builder.cc (left):

https://codereview.chromium.org/1153053005/diff/1/src/compiler/ast-graph-buil...
src/compiler/ast-graph-builder.cc:2228: if (expr->obj()->IsSuperReference()) {
Your last comment on the previous patch said you needed to add this. What
changed?

[arv (Not doing code reviews), 2015-05-26 21:03:46]

https://codereview.chromium.org/1153053005/diff/1/src/compiler/ast-graph-buil...
File src/compiler/ast-graph-builder.cc (left):

https://codereview.chromium.org/1153053005/diff/1/src/compiler/ast-graph-buil...
src/compiler/ast-graph-builder.cc:2228: if (expr->obj()->IsSuperReference()) {
On 2015/05/26 21:00:09, adamk wrote:
> Your last comment on the previous patch said you needed to add this. What
> changed?

I was mistaken... what was needed was the call to node->set_base_id in
VisitSuperReference.

[adamk, 2015-05-26 21:40:55]

lgtm

[arv (Not doing code reviews), 2015-05-28 18:14:42]

Remove a commented out loc that was left over

[arv (Not doing code reviews), 2015-05-28 18:15:05]

The CQ bit was checked by arv@chromium.org

[arv (Not doing code reviews), 2015-05-28 18:15:06]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/1153053005/#ps20001
(title: "Remove a commented out loc that was left over")

[commit-bot: I haz the power, 2015-05-28 18:15:22]

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

[commit-bot: I haz the power, 2015-05-28 18:50:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-28 18:51:02]

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