<font size="2"><font face="tahoma,sans-serif"><br></font></font><br><div class="gmail_quote">On Sat, Nov 6, 2010 at 10:37 AM, İsmail Dönmez <span dir="ltr"><<a href="mailto:ismail@namtrac.org">ismail@namtrac.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><font size="2"><font face="tahoma,sans-serif">Hi Chris;<br></font></font><br><div class="gmail_quote"><div class="im">

On Sat, Nov 6, 2010 at 8:18 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br><div><div>On Nov 3, 2010, at 3:54 PM, İsmail Dönmez wrote:</div>
<br><blockquote type="cite"><font size="2"><font face="tahoma,sans-serif">Hi;</font></font><div><font size="2"><font face="tahoma,sans-serif"><br></font></font></div><div><font size="2"><font face="tahoma,sans-serif">Original patch by pdox on #llvm . Asm parser was not handling fist and fistp instructions correctly. Please apply.</font></font></div>




</blockquote><br></div></div><div>Hi Ismail,</div><div><br></div><div>This is great detective work!  Just to verify my understanding, the issue is that GAS is treating:</div><div>   fistp (%rax)</div><div><div>as an alias for:</div>


<div>  fistps (%rax)</div><div>and we're compiling it to:</div></div><div>  fistpl (%rax)</div><div><br></div></div></blockquote><div><br></div></div><div>You are correct.</div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div style="word-wrap:break-word"><div></div><div>If this is the case, I think that the MC assembler should be changed to *reject* fisp, not to emulate GAS's behavior.  With a simple mem operand, there is nothing to say that "4 bytes" is the right size of the store, it should be diagnosed as an ambiguous instruction.  Reporting it as a bug in the code is much more friendly than miscompiling it of course. :)</div>


<div><br></div><div>What do you think?</div></div></blockquote><div><br></div></div><div>I am no assembly expert but this seems to be another vague gas behaviour. I guess rejecting this is the correct behaviour indeed.</div>

</div></blockquote><div><br></div><div>Also there is an existing alias for fstp, should that be removed also?</div><div><br></div><div>Regards,</div><div>ismail</div><div><br></div></div>