Request for review(XL): 6985015: C1 needs to support compressed oops

[Igor Veresov]

On 11/19/10 2:26 PM, Igor Veresov wrote:

>
>> src/cpu/x86/vm/c1_LIRGenerator_x86.cpp:
>>
>> 1143 if (!x->klass()->is_loaded() || LP64_ONLY(UseCompressedOops)
>> NOT_LP64(false)) {
>> 1165 if (!x->klass()->is_loaded() || LP64_ONLY(UseCompressedOops)
>> NOT_LP64(false)) {
>> I think the common pattern is LP64_ONLY(|| UseCompressedOops).
>
> It is, but do you think it would be more readable to factor this
> predicate to separate function. It just determines if we need an extra
> temporary. I'd rather leave it as is.
>

Sorry, I didn't get what you meant at first. Ramki explained it to me. 
Thanks, I'll fix that.

igor