From owner-freebsd-arm@FreeBSD.ORG  Mon Mar 18 11:13:34 2013
Return-Path: <owner-freebsd-arm@FreeBSD.ORG>
Delivered-To: freebsd-arm@freebsd.org
Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115])
 by hub.freebsd.org (Postfix) with ESMTP id 5BD9CCA1;
 Mon, 18 Mar 2013 11:13:34 +0000 (UTC)
 (envelope-from nick@flirble.org)
Received: from plum.flirble.org (plum.flirble.org [194.70.3.20])
 by mx1.freebsd.org (Postfix) with ESMTP id 2BE1FDE2;
 Mon, 18 Mar 2013 11:13:33 +0000 (UTC)
Received: from nick by plum.flirble.org with local (Exim 4.75 (FreeBSD))
 (envelope-from <nick@flirble.org>)
 id 1UHXhO-000LBx-LX; Mon, 18 Mar 2013 10:53:50 +0000
Date: Mon, 18 Mar 2013 10:53:50 +0000
From: Nicholas Clark <nick@ccl4.org>
To: Andrew Turner <andrew@fubar.geek.nz>
Subject: Re: RFC: Make clang the default compiler on ARM
Message-ID: <20130318105349.GS3729@plum.flirble.org>
References: <20130318220747.29d3fd29@bender>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20130318220747.29d3fd29@bender>
X-Organisation: Tetrachloromethane
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: Nicholas Clark <nick@flirble.org>
Cc: freebsd-arm@freebsd.org, freebsd-toolchain@freebsd.org
X-BeenThere: freebsd-arm@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: Porting FreeBSD to the StrongARM Processor <freebsd-arm.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-arm>,
 <mailto:freebsd-arm-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-arm>
List-Post: <mailto:freebsd-arm@freebsd.org>
List-Help: <mailto:freebsd-arm-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arm>,
 <mailto:freebsd-arm-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 18 Mar 2013 11:13:34 -0000

On Mon, Mar 18, 2013 at 10:07:47PM +1300, Andrew Turner wrote:
> I would like to make clang the default compiler on ARM using the patch
> at [1]. This only affects little-endian ARM as there is no support for
> big-endian ARM in clang.

> [1] http://people.freebsd.org/~andrew/clang/arm_clang_default.diff

I admit that I'm a complete outsider here I'm not sure if I'm qualified to
comment, but in the code in question, the comment above the changed line is
becoming increasingly incorrect, isn't it?

(This seems to reflect the sad reality of code comments generally, including
in the code I write and get to deal with.)

Nicholas Clark