Date: Mon, 18 Mar 2013 11:36:39 -0400 From: Eitan Adler <lists@eitanadler.com> To: Nicholas Clark <nick@ccl4.org> Cc: freebsd-arm@freebsd.org, freebsd-toolchain@freebsd.org Subject: Re: RFC: Make clang the default compiler on ARM Message-ID: <CAF6rxgn%2By4tQzBC58uTUQ3PUe6gT1pudnMXOcSBUeEVxR6qniA@mail.gmail.com> In-Reply-To: <20130318105349.GS3729@plum.flirble.org> References: <20130318220747.29d3fd29@bender> <20130318105349.GS3729@plum.flirble.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 18 March 2013 06:53, Nicholas Clark <nick@ccl4.org> wrote: > 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? I think you are right. Given that the comment duplicates the if statement below, it should probably just be dropped. -- Eitan Adler
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxgn%2By4tQzBC58uTUQ3PUe6gT1pudnMXOcSBUeEVxR6qniA>