(XS) RFR: 8077674: BSD build failures due to undefined macros

David Holmes david.holmes at oracle.com
Thu Apr 16 01:50:17 UTC 2015


Still need Review for this please (with plans to backport to 8u).

Thanks,
David

On 15/04/2015 6:25 AM, David Holmes wrote:
> Hi Dmitry,
>
> On 14/04/2015 9:25 PM, Dmitry Samersoff wrote:
>> David,
>>
>> It's better to change
>>
>> #ifdef __APPLE__
>>
>> to
>>
>> #if defined(__APPLE__)
>
> Why? They seem 100% equivalent. Plus that would mean a lot of changes.
> Plus we use that pattern in many, many, many places.
>
> Thanks,
> David
>
>> -Dmitry
>>
>>
>> On 2015-04-14 13:20, David Holmes wrote:
>>> webrev:
>>>
>>> http://cr.openjdk.java.net/~dholmes/8077674/webrev/
>>>
>>>  From the original error report:
>>>> "-Wundef option causes GCC to warn whenever it encounters an identifier
>>>> which is not a macro in an ‘#if’. "
>>>>
>>>> Hence we should be using
>>>>
>>>> #elif defined(__FreeBSD__)
>>>>
>>>> etc in this context.
>>>
>>> I also founds similar suspect usages in os_linux.cpp and os_windows.cpp.
>>>
>>> Thanks,
>>> David
>>
>>


More information about the hotspot-runtime-dev mailing list