Request for review (XS): 7036960: TemplateTable::fast_aldc in templateTable_x86_64.cpp uses movptr instead of load_klass
Christian Thalinger
christian.thalinger at oracle.com
Tue Apr 19 01:33:22 PDT 2011
On Apr 18, 2011, at 7:39 PM, Tom Rodriguez wrote:
> 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.
I agree, we should do that. -- Christian
>
> 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