review for 7090654: nightly failures after 7086585
Vladimir Kozlov
vladimir.kozlov at oracle.com
Wed Sep 14 13:31:36 PDT 2011
Looks good.
We should also fix error message to print actual index and class name:
fatal error: Invalid constant pool index %u for field name in class file %s
But it could wait.
Thanks,
Vladimir
Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/7090654
> 21 lines changed: 0 ins; 6 del; 15 mod; 2496 unchg
>
> 7090654: nightly failures after 7086585
> Reviewed-by:
>
> I had made late changes to accessing of fields in InstanceKlass and
> didn't rerun my tests so I create a mismatch between the indexing.
> Some code was using i * 7 and others was using i. I've corrected it
> so it's using i consistently everything. Tested with sajdi.
>
> I screwed up the new of fields when we recreate a .class file from our
> internal representation. It should be unscaled. Tested with
> java/lang/instrument regression tests. I had thought the tests I
> originally ran exercised this path but I was wrong.
>
More information about the hotspot-compiler-dev
mailing list