[9] RFR(L): 8034812: remove IDX_INIT macro hack in Node class

[Vladimir Kozlov]

Thank you, Tobias, for this testing. I remove my concern. Changes are good.

Thanks,
Vladimir

On 5/28/14 12:42 AM, Tobias Hartmann wrote:
> Hi Vladimir,
>
> as you suggested, I tested the fix both on x86 and sparc by running Octane with -Xcomp and -XX:-TieredCompilation and
> measured the compilation speed. There is no regression. I attached the results [1] to the bug.
>
> I also executed SPECjbb2005, SPECjbb2013 and SPECjvm2008-Compiler on Perf-Aurora to make sure that there is no
> performance regression. I posted the link in the bug description.
>
> Thanks,
> Tobias
>
>
> [1]
> -- x86_64, Solaris --
>
> Baseline
>    Score: 150.8 (+- 2.1)
>    Compilation speed: 31548.7 bytes/s (+- 247.67)
> Fix
>    Score: 151.1 (+- 1.86)
>    Compilaion speed: 31878.5 bytes/s (+- 195.88)
>
>
> -- Sparc, Solaris --
>
> Baseline
>    Score: 17.46 (+- 0.11)
>    Compilation speed: 4342.5 (+- 55.32)
> Fix
>    Score: 17.4 (+- 0.13)
>    Compilation speed: 4349.5 (+- 69.66)
>
>
> The numbers in brackets show the 95% confidence interval.
>
>
> On 15.05.2014 23:27, Vladimir Kozlov wrote:
>> Tobias,
>>
>> Changes are fine but I am concern about compilation speed.
>> Please, test compilation performance (-XX:+TimeCompiler) on x86 and sparc. You need to build optimized VM for that to
>> exclude assert but keep statistic code. I would suggest to run one of nashorn octane benchmarks with -Xcomp
>> -XX:-TieredCompilation to get compilation with big graphs.
>>
>> Thanks,
>> Vladimir
>>
>> On 5/15/14 11:53 AM, Tobias Hartmann wrote:
>>> Hi,
>>>
>>> please review the following patch that removes the IDX_INIT macro used
>>> by Node::Node(...) to retrieve the Compile object and replaces it by a
>>> call to Compile::current(). The Node constructor, new operator and all
>>> calls to it are adapted accordingly.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8034812
>>> Webrev: http://cr.openjdk.java.net/~anoll/8034812/webrev.00/
>>>
>>> Tested with JPRT.
>>>
>>> Thanks,
>>> Tobias
>