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