Request for review (XS): 7036960: TemplateTable::fast_aldc in templateTable_x86_64.cpp uses movptr instead of load_klass

Tom Rodriguez tom.rodriguez at oracle.com
Mon Apr 18 10:39:25 PDT 2011


We should consider hiding klass_offset_in_bytes from most code in the system so that we are forced to use the appropriate MacroAssembler wrappers.  We will probably need to add some more wrappers to cover everything.  I wouldn't be surprised if there are more problems like this lurking.

tom

On Apr 15, 2011, at 4:49 AM, Christian Thalinger wrote:

> http://cr.openjdk.java.net/~twisti/7036960/
> 
> 7036960: TemplateTable::fast_aldc in templateTable_x86_64.cpp uses movptr instead of load_klass
> Reviewed-by:
> 
> The fix is to replace movptr with load_klass.
> 
> Additionally I changed all occurrences of movptr instructions loading
> the class of an oop with load_klass in templateTable_x86_32.cpp (to be
> some consistent).
> 
> src/cpu/x86/vm/templateTable_x86_32.cpp
> src/cpu/x86/vm/templateTable_x86_64.cpp
> 



More information about the hotspot-compiler-dev mailing list