review request (XS): 6996563: 6984311 changes forgot to update vmStructs.cpp for new field _operands

John Rose john.r.rose at oracle.com
Fri Nov 5 00:12:29 PDT 2010


I did find one pre-existing bug which prevented me from validating this fix.  See updated webrev.

With this change, I can use the SA "dumpclass" command to create a dumped class file which has the same contents (except for stackmaps) and can be loaded by the JVM in place of the original file.

http://cr.openjdk.java.net/~jrose/6996563/webrev.01

-- John

On Nov 3, 2010, at 1:05 AM, Christian Thalinger wrote:

> On Nov 3, 2010, at 7:00 AM, John Rose wrote:
>> Bug summary says it all.
>> 
>> http://cr.openjdk.java.net/~jrose/6996563/webrev/
>> 
>> I may update this webrev if I find other problems during testing.
> 
> Looks good.  -- Christian



More information about the hotspot-compiler-dev mailing list