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

Renato Golin renato.golin at linaro.org
Fri Dec 20 13:49:51 PST 2013

On 20 December 2013 21:48, David Peixotto <dpeixott at codeaurora.org> wrote:

> Yes, storing the operand as a float is actually required to get the
> parsing to work correctly. In the ARMOperand::isFPImm() function it
> converts the operand to a float and checks to see if it can be encoded as
> an 8-bit operand. When we kept the parsed value as a double, this check
> failed because the lower 32-bits of the double was zero so it looked like
> the float 0.0f which cannot be encoded by the 8-bit encoding.


This code is a white list of instructions that end in 's' but do not
> actually set the flags so fconstd is not needed here.

Ah, ok.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131220/a76c54ef/attachment.html>

More information about the llvm-commits mailing list