[turbofan] Improve inline-exception tests.

[turbofan] Improve inline-exception tests.

- Make constants more interesting.
- Add an addition to be done after the inlined call in the try-block.
- On command line, have a bit more output.
- New alternative that deopts from unoptimized code.

BUG=
R=jarin

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

[bgeron, 2016-08-26 14:44:35]

The CQ bit was checked by bgeron@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-26 14:44:48]

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

[bgeron, 2016-08-26 14:47:06]

Please take a look.

This adds tests such as the following:

    f = function f____2_t___c_l____ld () {
      var local = 888;
      deopt = true;
      try {
        counter++;
        return 4 + increaseAndThrow42_noopt_outer();
        counter++;
      } catch (ex) {
        counter++;
        local += ex;
        counter++;
      }
      counter++;
      return 5 + local;
    }
    resetOptAndAssertResultEquals(939, f);
    assertEquals(5, counter);



    function increaseAndThrow42_noopt_inner() {
      if (deopt) %DeoptimizeFunction(f);
      counter++;
      throw 42;
    }

    %NeverOptimizeFunction(increaseAndThrow42_noopt_inner);

    function increaseAndThrow42_noopt_outer() {
      return increaseAndThrow42_noopt_inner();
    }

[commit-bot: I haz the power, 2016-08-26 15:04:27]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-26 15:04:28]

Dry run: This issue passed the CQ dry run.

[Jarin, 2016-08-29 09:48:07]

lgtm

https://codereview.chromium.org/2285743002/diff/1/test/mjsunit/compiler/inlin...
File test/mjsunit/compiler/inline-exception-1.js (right):

https://codereview.chromium.org/2285743002/diff/1/test/mjsunit/compiler/inlin...
test/mjsunit/compiler/inline-exception-1.js:101: function
increaseAndThrow42_noopt_outer() {
The "noopt" in the name is confusing because increaseAndThrow42_noopt_outer will
be inlined and optimized.

[bgeron, 2016-08-29 09:51:47]

How about increaseAndThrow42_calls_noopt?

On 2016/08/29 09:48:07, Jarin wrote:
> lgtm
> 
>
https://codereview.chromium.org/2285743002/diff/1/test/mjsunit/compiler/inlin...
> File test/mjsunit/compiler/inline-exception-1.js (right):
> 
>
https://codereview.chromium.org/2285743002/diff/1/test/mjsunit/compiler/inlin...
> test/mjsunit/compiler/inline-exception-1.js:101: function
> increaseAndThrow42_noopt_outer() {
> The "noopt" in the name is confusing because increaseAndThrow42_noopt_outer
will
> be inlined and optimized.

[Jarin, 2016-08-29 10:18:47]

On 2016/08/29 09:51:47, bgeron wrote:
> How about increaseAndThrow42_calls_noopt?

Sounds good.

[bgeron, 2016-08-29 10:51:22]

The CQ bit was checked by bgeron@google.com

[bgeron, 2016-08-29 10:51:22]

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/2285743002/#ps20001
(title: "Rename noopt function. Generate some type feedback for noopt
functions.")

[commit-bot: I haz the power, 2016-08-29 10:51:31]

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-29 11:16:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-29 11:17:24]

Description was changed from

==========
[turbofan] Improve inline-exception tests.

- Make constants more interesting.
- Add an addition to be done after the inlined call in the try-block.
- On command line, have a bit more output.
- New alternative that deopts from unoptimized code.

BUG=
R=jarin
==========

to

==========
[turbofan] Improve inline-exception tests.

- Make constants more interesting.
- Add an addition to be done after the inlined call in the try-block.
- On command line, have a bit more output.
- New alternative that deopts from unoptimized code.

BUG=
R=jarin

Committed: https://crrev.com/d90d76bdb5bfa095adfb95c445953660bbf6d5e8
Cr-Commit-Position: refs/heads/master@{#38974}
==========

[commit-bot: I haz the power, 2016-08-29 11:17:25]

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