[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Re: RFR: 8019470 "Changes needed to compile JDK8 on MacOS with clang compiler"
From:       David Holmes <david.holmes () oracle ! com>
Date:       2013-07-25 4:44:13
Message-ID: 51F0AD1D.1080907 () oracle ! com
[Download RAW message or body]

On 25/07/2013 8:48 AM, Tim Bell wrote:
> 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

Yes but that doesn't explain the change now needed to os_bsd.cpp in hotspot.

David
-----

>
> Tim
>
>
> On 07/24/13 02:59 PM, Christian Thalinger wrote:
>> On Jul 24, 2013, at 5:15 AM, Tim Bell <tim.bell@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
>>>>>
>
[prev in list] [next in list] [prev in thread] [next in thread]