[PATCH] Implements -mfpu=softvfp+variants on ARM driver

Renato Golin renato.golin at linaro.org
Tue Feb 18 03:56:19 PST 2014


On 18 February 2014 10:53, Bernie Ogden <bogden at arm.com> wrote:
> Nitpick: line 724 of Tools.cpp is blank.

Done.


> You're testing that we're setting the right target features for FP hardware,
> but I don't see tests that we're getting the right calling convention. Is
> that something that could reasonably be added, or am I just showing my
> ignorance here?

No, I completely forgot. Thanks! ;)


> Is float-abi an exclusively ARM thing?

I don't think so, though it probably more important in ARM than other archs.


> And do we care about cross-target CLI consistency?

We do, unfortunately. It's a lot easier to cross-compile using the
same build system.


> Basically LGTM, modulo the nitpick and the tests.

Thanks!

--renato



More information about the cfe-commits mailing list