<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 20 December 2013 21:48, David Peixotto <span dir="ltr"><<a href="mailto:dpeixott@codeaurora.org" target="_blank">dpeixott@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><span style="color:rgb(34,34,34)">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.</span></div>
</blockquote><div><br></div><div>Right.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><span style="color:rgb(34,34,34)">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.</span></div>
</blockquote><div><br></div><div>Ah, ok.</div><div><br></div><div>cheers,<br></div><div>--renato </div></div><br></div></div>