[cfe-commits] [Patch] Initial FreeBSD ARM support

[Andrew Turner]

I have attached a new patch that updates the tests.

On Fri, 7 Dec 2012 09:24:08 -0500
Rafael Espíndola <rafael.espindola at gmail.com> wrote:

> > I have reviewed this patch (over three iterations) and am happy
> > with it.  I'd be happy to commit it if none of the driver people
> > want to give it another look.
> 
> You pass -mfpu=softvfp even when you have a hard float abi
> (llvm::Triple::GNUEABIHF), no? Is that really correct?
There is currently no support in FreeBSD for any of the the ARM EABI
variants or for hard float. I am working on adding support to FreeBSD
for the ARM EABI but it is not yet in the main development tree. I
included the EABI changes so, when this support is included, we will
not need to update clang again to use it.

For hard float we have support for vfp in the kernel however it is not
enabled in any of the kernel configs and I'm not sure about the state
of it. Because of this I consider llvm::Triple::GNUEABIHF to be
unsupported on FreeBSD.

I can update the code to handle the llvm::Triple::GNUEABIHF however it
will be untested and unable to be used on any FreeBSD system. I intend
to fix this when FreeBSD supports the ARM EABI however until then I
would like to get clang working in the currently supported and soon to
be supported ABIs.

> 
> test/Misc/freebsd-arm-size_t.c:
> 
> Nit: Use "clang -cc1" and -verify instead of "clang -Werror".
> 
> Please make sure that you have a test where  -mfpu=softvfp shows up.
> Please add a test with the hard float environment.

I've updated the tests to ensure -mfpu=softvfp shows up, and changed it
to use %clang_cc1.

Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang_arm_6.diff
Type: text/x-patch
Size: 4262 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121209/672a1654/attachment.bin>