[llvm-commits] patch: use movd for 32-bits and movq for 64-bits consistently

[Nick Lewycky]

On 16 June 2011 15:37, Eli Friedman <eli.friedman at gmail.com> wrote:

> On Thu, Jun 16, 2011 at 3:35 PM, Nick Lewycky <nlewycky at google.com> wrote:
> > On 16 June 2011 15:29, Eli Friedman <eli.friedman at gmail.com> wrote:
> >>
> >> On Thu, Jun 16, 2011 at 2:33 PM, Anton Korobeynikov <asl at math.spbu.ru>
> >> wrote:
> >> >> IIRC, we do things the way we do (outside of parsing asm) because the
> >> >> old Darwin assembler wouldn't accept "movq %rax, %xmm0", and required
> >> >> "movd %rax, %xmm0" instead.  I don't recall all the details of that,
> >> >> though.
> >> > Yeah. Probably it's safe nowadays to get rid of this workaround?
> >>
> >> Note that even if we do decide to get rid of that workaround, we still
> >> need to do the right thing with "movd %rax, %xmm0".
> >
> > Reject it? Or accept it and assemble as "movq %rax, %xmm0" like GNU as
> > does? We've been pretty aggressive about rejecting bad assembly,
> regardless
> > of what as does.
>
> Accept it; it would be weird to reject a construct that LLVM itself
> has been generating for a very long time.
>

Sound reasonable. Patch attached!

Nick
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110616/a9d726e1/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: movd-movq.patch
Type: text/x-patch
Size: 9098 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110616/a9d726e1/attachment.bin>