RFR: 8019470 "Changes needed to compile JDK8 on MacOS with clang compiler"

[Tim Bell]

Christian:

> That's not true.  I've added Mac OS X support with the same change.

For building hotspot only, perhaps.  I want to build the entire product, 
start to finish, using clang.

That's why I needed to touch these files:

common/autoconf/hotspot-spec.gmk.in
common/autoconf/toolchain.m4
make/jprt.properties


Tim


On 07/24/13 02:59 PM, Christian Thalinger wrote:
> On Jul 24, 2013, at 5:15 AM, Tim Bell <tim.bell at oracle.com> wrote:
>
>> On 07/24/13 04:43 AM, David Holmes wrote:
>>> Hi Tim,
>>>
>>> On 24/07/2013 8:15 AM, Tim Bell wrote:
>>>> Hello everyone-
>>>>
>>>> This is a small set of changes to switch from gcc to clang when building
>>> So we already added support for clang as the compiler for hotspot - is this just extending things to allow configure to select clang?
>> The earlier clang/hotspot activity was centered around Linux x86/x86_64.
> That's not true.  I've added Mac OS X support with the same change.
>
> -- Chris
>
>>>> on MacOS, and also enable building on MacOS 8.x
>>> Have we already updated all the JPRT queues to have 10.8 macs versus 10.7 ones?
>> Yes, all the queues have 10.8 Macs, which have been test-only up until now.
>>
>>>> Here is the bug report:
>>>>
>>>> http://bugs.sun.com/view_bug.do?bug_id=8019470
>>>>
>>>> Here are the webrevs:
>>>>
>>>> Hotspot only:
>>>>
>>>>    http://cr.openjdk.java.net/~tbell/8019470/hotspot/webrev.00/
>>> src/os/bsd/vm/os_bsd.cpp
>>>
>>> Again I'm confused. We already allowed building via clang so why wasn't this change needed before?
>> This file is not used when building on Linux.
>>
>> Tim
>>
>>> Thanks,
>>> David
>>>
>>>> All of the changes:
>>>>
>>>>    http://cr.openjdk.java.net/~tbell/8019470/webrev.00/
>>>
>>>> Thanks in advance.
>>>>
>>>> Tim
>>>>