[turbofan] Fix JSInliner strong mode bailout.

[turbofan] Fix JSInliner strong mode bailout.

This moves the bailout point in the JSInliner up to a point where it is
still allowed to decide not to inline. Once the inlining decision has
been recorded with CompilationInfo::AddInlinedFunction, we should not
abort anymore.

R=jarin@chromium.org

Committed: https://crrev.com/4054961fa43c5ad85b25badb63c9ed7a93d1fafa
Cr-Commit-Position: refs/heads/master@{#31469}

[Jarin, 2015-10-22 09:42:17]

lgtm (once the signedness problem is resolved)

[Jarin, 2015-10-22 10:03:09]

still lgtm.

https://codereview.chromium.org/1410023006/diff/20001/src/compiler/js-inlinin...
File src/compiler/js-inlining.cc (left):

https://codereview.chromium.org/1410023006/diff/20001/src/compiler/js-inlinin...
src/compiler/js-inlining.cc:405: size_t const inlinee_formal_parameters =
start->op()->ValueOutputCount() - 3;
Now I am wondering whether we should dcheck that 

start->op()->ValueOutputCount() - 3 == parameter_count.

Maybe not (I do not even know why "3" is there; I am guessing one is the
receiver).

[Michael Starzinger, 2015-10-22 10:35:41]

https://codereview.chromium.org/1410023006/diff/20001/src/compiler/js-inlinin...
File src/compiler/js-inlining.cc (left):

https://codereview.chromium.org/1410023006/diff/20001/src/compiler/js-inlinin...
src/compiler/js-inlining.cc:405: size_t const inlinee_formal_parameters =
start->op()->ValueOutputCount() - 3;
On 2015/10/22 10:03:09, Jarin wrote:
> Now I am wondering whether we should dcheck that 
> 
> start->op()->ValueOutputCount() - 3 == parameter_count.
> 
> Maybe not (I do not even know why "3" is there; I am guessing one is the
> receiver).
> 

Done.

[Michael Starzinger, 2015-10-22 11:40:23]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2015-10-22 11:40:24]

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/1410023006/#ps40001
(title: "Addressed comment.")

[commit-bot: I haz the power, 2015-10-22 11:40:27]

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

[commit-bot: I haz the power, 2015-10-22 11:41:58]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-22 11:42:26]

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