[cfe-dev] compiler-rt non-neon builds are still broken

[İsmail Dönmez]

Hi Renato,

On Sun, Dec 14, 2014 at 6:43 PM, Renato Golin <renato.golin at linaro.org> wrote:
> On 14 December 2014 at 14:02, İsmail Dönmez <ismail at donmez.ws> wrote:
>> Scratch that, doesn't seem to work for my cross-build setup. But seems
>> to work on native ARM builder. Weird.
>
> Hi Ismail,
>
> @ is the comment character in ARM assembly, but not always recognised? Not sure.

Nope it was just bad testing. It failed on native build too.

> Anyway, we should not even try to unwind D16~D31 when you compile the
> library with VFPv3-D16. If we had push/pop for .cpu and .fpu for ARM,
> we could fix that one pretty quickly, but we don't.
>
> My guess is that we should not be emitting that line at all, so
> replacing __ARM_FP by __ARM_NEON would be the right thing, and only
> emitting stcl when ARCH < 7 and vstmia otherwise.
> If that works, it'll depend on how you compile your library, and it'd
> be good to know how people do that and what do they expect to see. For
> instance, I don't expect to see that function be called at all when
> ARCH < 7 && !NEON.

This might be true but gcc compiles this fine so at least there might
be an ASM parser bug.

Thanks,
ismail