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.

Looks good to me.  I will add the changes and push this one.

-- Christian

More information about the hotspot-compiler-dev mailing list