svn commit: r291937 - in head: lib/libc/aarch64/sys lib/libc/arm/sys sys/arm/arm sys/arm/include sys/arm64/arm64 sys/arm64/include sys/conf sys/kern

[Ian Lepore]

On Thu, 2015-12-24 at 23:10 +0200, Konstantin Belousov wrote:
> On Thu, Dec 24, 2015 at 01:18:53PM -0700, Ian Lepore wrote:
> > Oh, I know what's likely at the heart of this... I'm using gcc
> > 4.2.1
> > for arm v4/v5, because clang 3.7 is broken (works to crossbuild,
> > but
> > fails to run native).  The mrrc instruction was introduced at arm
> > arch
> > 5E, I'll bet clang is defaulting to 5E and gcc defaults to 4.
> > 
> > This is what's in my make.conf for the build that failed:
> > 
> > WITH_GCC=yes
> > WITH_GNUCXX=yes
> > WITH_GCC_BOOTSTRAP=yes
> > WITHOUT_CLANG=yes
> > WITHOUT_CLANG_IS_CC=yes
> > WITHOUT_CLANG_BOOTSTRAP=yes
> > 
> > Yep, just confirmed it, switched back to clang 3.7 for crossbuild
> > and
> > no errors.
> 
> I think that the following is the least intrusive change.  I built it
> with your make.conf successfully (and make.conf seems to take effect
> judging
> by the build time).
> 
> If you are fine with the change, I will commit right after confirming
> that
> ARMv6 build still results in correct code (building right now).

yeah, gcc sure builds fast compared to clang. :)

The change looks good to me.

-- Ian