[turbofan]: Fix tail calls edge cases and add tests

[turbofan]: Fix tail calls edge cases and add tests

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

[danno, 2015-07-20 15:21:51]

danno@chromium.org changed reviewers:
+ titzer@chromium.org

[danno, 2015-07-20 15:21:58]

PTAL

[danno, 2015-07-21 11:44:39]

danno@chromium.org changed reviewers:
+ bmeurer@chromium.org

[titzer, 2015-07-21 11:48:52]

https://codereview.chromium.org/1245523002/diff/100001/test/unittests/compile...
File test/unittests/compiler/linkage-tail-call-unittest.cc (right):

https://codereview.chromium.org/1245523002/diff/100001/test/unittests/compile...
test/unittests/compiler/linkage-tail-call-unittest.cc:53:
LocationSignature::Builder locations1(zone(), 1, 0);
You don't have to use the Signature::Builder every time; you can just have a
static array and pass it directly to the constructor of Signature.

[danno, 2015-07-21 14:19:05]

On 2015/07/21 at 11:48:52, titzer wrote:
>
https://codereview.chromium.org/1245523002/diff/100001/test/unittests/compile...
> File test/unittests/compiler/linkage-tail-call-unittest.cc (right):
> 
>
https://codereview.chromium.org/1245523002/diff/100001/test/unittests/compile...
> test/unittests/compiler/linkage-tail-call-unittest.cc:53:
LocationSignature::Builder locations1(zone(), 1, 0);
> You don't have to use the Signature::Builder every time; you can just have a
static array and pass it directly to the constructor of Signature.

I updated the tests, they use your suggestion and should be more compact now.
Please take another look.

[titzer, 2015-07-22 15:22:19]

lgtm

[danno, 2015-07-22 15:27:24]

The CQ bit was checked by danno@chromium.org

[commit-bot: I haz the power, 2015-07-22 15:27:39]

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

[commit-bot: I haz the power, 2015-07-22 18:27:21]

Committed patchset #12 (id:220001)

[commit-bot: I haz the power, 2015-07-22 18:27:41]

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