RFR(M): 8007439 : C2: adding successful message of inlining

[Vladimir Kozlov]

This look good. Thanks!

We do not use 'mutable' in our sources. Why you need it?

In next code you can also use local fail_msg instead of overwriting _msg 
field:

!     if (max_inline_size > default_max_inline_size) {
!       set_msg("hot method too big");
       }
!     set_msg("too big");
!     return false;

Thanks,
Vladimir

On 2/7/13 9:49 AM, Igor Ignatyev wrote:
> I have update the webrev.
>
>   - added _msg field  with getter and setter
>   - removed parameter 'msg' from InlineTree::print_inlining()
>   - methods InlineTree::should_inline(), InlineTree::should_not_inline()
> are no longer const-method
>
> http://cr.openjdk.java.net/~iignatyev/8007439/webrev.01/
>
>
> Best regards,
> Igor Ignatyev
>
> On 02/06/2013 06:20 AM, Vladimir Kozlov wrote:
>> Add _msg field into InlineTree class with getter and setter instead of
>> passing is as parameter.
>>
>> Thanks,
>> Vladimir
>>
>> On 2/5/13 9:46 AM, Igor Ignatyev wrote:
>>> Hi all,
>>>
>>> Please review the patch.
>>>
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8007439/webrev.00/
>>> testing: JPRT