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

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Nov 5 09:19:38 PDT 2010


Looks good.

Vladimir

John Rose wrote:
> 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