Request for reviews (XXL): 6893268: additional dynamic language related optimizations in C2
Christian Thalinger
Christian.Thalinger at Sun.COM
Mon Jan 4 01:17:59 PST 2010
On Fri, 2009-12-18 at 23:00 -0800, John Rose wrote:
> I made the changes you suggested on top of Christian's. I also
> removed the 'friend ciKlass' declaration.
Thank you.
> But I don't think the package test was strict enough. I added checks
> to reject the klass if it is in a subpackage of the queried package,
> or if it is not loaded by the system class loader. I also moved the
> logic into ciInstanceKlass, where I think it fits better.
>
> Doing the subpackage test requires an "index_of" operation in
> symbolOop etc.
>
> Here's the patch; let me know what you think. I could hold back this
> change until I do the "trust finals" option in a later push.
>
> http://cr.openjdk.java.net/~jrose/pkgtest/webrev.00/
Looks good to me. I will add the changes and push this one.
-- Christian
More information about the hotspot-compiler-dev
mailing list