RFR: 8240846: Zero VM is broken after JDK-8238681: UseSSE not defined
Robbin Ehn
robbin.ehn at oracle.com
Fri Mar 13 08:20:49 UTC 2020
On 2020-03-13 09:12, Aleksey Shipilev wrote:
> On 3/13/20 1:31 AM, jiefu(傅杰) wrote:
>> diff -r 15d69d370743 src/hotspot/share/compiler/compileBroker.cpp
>> --- a/src/hotspot/share/compiler/compileBroker.cpp Thu Mar 12 16:05:07 2020 -0700
>> +++ b/src/hotspot/share/compiler/compileBroker.cpp Fri Mar 13 08:29:26 2020 +0800
>> @@ -1328,7 +1328,7 @@
>> // do the compilation
>> if (method->is_native()) {
>> if (!PreferInterpreterNativeStubs || method->is_method_handle_intrinsic()) {
>> -#ifdef X86
>> +#if defined(X86) && !defined(ZERO)
>> // The following native methods:
>> //
>> // java.lang.Float.intBitsToFloat
>> @@ -1350,7 +1350,7 @@
>> method->intrinsic_id() == vmIntrinsics::_doubleToRawLongBits))) {
>> return NULL;
>> }
>>
>> -#endif // X86
>> +#endif // X86 && !ZERO
>> // To properly handle the appendix argument for out-of-line calls we are using a small trampoline that
>> // pops off the appendix argument and jumps to the target (see gen_special_dispatch in SharedRuntime).
>> ------------------------------------
>
> The fix came off badly formatted, please consider webrev next time?
>
> I believe this addition is safe, as Zero is not supposed to call that method at all? If so, looks good!
>
+1, thanks.
/Robbin
More information about the hotspot-compiler-dev
mailing list