Request for reviews (L): 6939203: JSR 292 needs method handle constants

Tom Rodriguez tom.rodriguez at oracle.com
Wed Jun 2 16:02:14 PDT 2010


I suspect it just wasn't exercised, otherwise I think you would have hit this assert:

            assert(load_klass() != NULL, "klass not set");

You can stress all C1's patching logic using -XX:+PatchALot.  That forces every possible patch site to be patched.

tom

On Jun 2, 2010, at 3:29 PM, John Rose wrote:

> On Jun 2, 2010, at 3:24 PM, Tom Rodriguez wrote:
> 
>> So shouldn't it be?
>> 
>> +          k = cc->resolve_constant(CHECK);
> 
> (Smacks forehead.)  Right.
> 
> How did that ever work?  I'm going to recheck my testing...
> 
> -- John



More information about the hotspot-compiler-dev mailing list