RFR (XS): 8043210: Add _BIG_ENDIAN define on linux/sparc

Mikael Vidstedt mikael.vidstedt at oracle.com
Thu May 22 20:38:42 UTC 2014


AFAICT PPC already has the define:

  199 CFLAGS_REQUIRED_ppc     += -fsigned-char -D_BIG_ENDIAN

Cheers,
Mikael

On 2014-05-22 13:31, Kumar Srinivasan wrote:
> Looks good but IIRC PPC also needs this.
>
> Kumar
>
> On 5/22/2014 1:13 PM, Mikael Vidstedt wrote:
>>
>> Please review the following change, destined for jdk7u-dev. The zip 
>> implementation in unpack200 depends on _BIG_ENDIAN being defined on 
>> big endian platforms, and the define is missing on the linux/sparc 
>> platform(s).
>>
>> Note that in the new build system (jdk8+) the relevant logic was 
>> rewritten, so the problem does not exists there.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8043210
>> Webrev: 
>> http://cr.openjdk.java.net/~mikael/webrevs/8043210/webrev.00/webrev/
>>
>> Thanks,
>> Mikael
>>
>



More information about the jdk7u-dev mailing list