On Jul 24, 2013, at 5:15 AM, Tim Bell <tim.b...@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 >>> >