[llvm-commits] PATCH: Fix PR8528

[İsmail Dönmez]

Hi;

On Sat, Nov 6, 2010 at 8:32 PM, Chris Lattner <clattner at apple.com> wrote:
> On Nov 6, 2010, at 1:42 AM, İsmail Dönmez wrote:
> Also there is an existing alias for fstp, should that be removed also?
>
> Yes, you're completely right!  I fixed this in r118346, we now get:
> t.s:1:1: error: ambiguous instructions require an explicit suffix (could be
> 'fstps', 'fstpl', or 'fstpt')
> fstp (%rax)
> ^
> t.s:2:1: error: ambiguous instructions require an explicit suffix (could be
> 'fistps', or 'fistpl')
> fistp (%rax)
> ^
> t.s:3:1: error: ambiguous instructions require an explicit suffix (could be
> 'fists', or 'fistl')
> fist (%rax)
> ^
> Thanks again for tracking this down!

Thanks for fixing this, I submitted a patch for MPlayer to use the
correct instructions.

Regards,
ismail