[9] RFR(L): 8034812: remove IDX_INIT macro hack in Node class
Vladimir Kozlov
vladimir.kozlov at oracle.com
Thu May 15 21:27:32 UTC 2014
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
More information about the hotspot-compiler-dev
mailing list