[PATCH] Allow encoded 8-bit floating point constants in ARM vmov instructions

[Renato Golin]

On 18 December 2013 18:28, David Peixotto <dpeixott at codeaurora.org> wrote:

> Both gcc and armasm accept that syntax for the fconstd/fconsts
> instructions, but reject it for the vmov.f* functions.
>

I'm lost. I thought that we were discussing this because another assembler
(GNU or ARM) accepted this syntax, while LLVM didn't. If the other
assemblers don't accept the syntax, how did you find that code in the first
place?

There is a possibility I'm really lost, and we're not talking about GNU
syntax any more, but some nice things to have. In that case, I'd vote
against, since I don't want to create the same problem to other compilers
that they created for us.

If neither GAS nor ARMASM support encoded FPs on vmov.f, I think we should
refuse with a similar error message.

If the encoded FP problem stops us from supporting another instructions, we
should fix that.

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131218/1d26d7e5/attachment.html>