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

[John Rose]

On May 15, 2014, at 2:27 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:

> 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.

A little software archeology brings up some of the history behind the "new(N,C)" hack:


D 1.156 05/06/29 16:29:14 vkozlov 258 257
6219945: C2 virtual methods cleanup. Reduce calls Compile::current().
https://bugs.openjdk.java.net/browse/JDK-6219945

So it's been 9 years; do we still want to reduce calls to Compile::current()?  As a wild guess, it could cause a 1% regression in compile speed.

— John
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140516/da794fb9/attachment-0001.html>