X-Git-Url: http://demsky.eecs.uci.edu/git/?a=blobdiff_plain;f=lib%2FTarget%2FX86%2FREADME-MMX.txt;h=a6c8616b6d2c6b77a897de92552ef3111b20d25b;hb=d1474d09cbe5fdeec8ba0d6c6b52f316f3422532;hp=7ad623e40d51dca4e4712383d23e6f4c6e8391be;hpb=ea1cddf546474ba167bc4c83f8afbb6f85326a7e;p=oota-llvm.git diff --git a/lib/Target/X86/README-MMX.txt b/lib/Target/X86/README-MMX.txt index 7ad623e40d5..a6c8616b6d2 100644 --- a/lib/Target/X86/README-MMX.txt +++ b/lib/Target/X86/README-MMX.txt @@ -41,3 +41,31 @@ _qux: addl $12, %esp ret $4 +//===---------------------------------------------------------------------===// + +We generate crappy code for this: + +__m64 t() { + return _mm_cvtsi32_si64(1); +} + +_t: + subl $12, %esp + movl $1, %eax + movd %eax, %mm0 + movq %mm0, (%esp) + movl (%esp), %eax + movl 4(%esp), %edx + addl $12, %esp + ret + +The extra stack traffic is covered in the previous entry. But the other reason +is we are not smart about materializing constants in MMX registers. With -m64 + + movl $1, %eax + movd %eax, %mm0 + movd %mm0, %rax + ret + +We should be using a constantpool load instead: + movq LC0(%rip), %rax