From acb5310542b82e7acf5f5a91fa619e4b7efeff5c Mon Sep 17 00:00:00 2001 From: Bill Wendling Date: Thu, 22 Mar 2012 23:25:07 +0000 Subject: [PATCH] When an MMX output variable is tied to the input variable, we have to implicitly cast the value to x86_mmx. This gives the ASM string the correct call signature. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@153290 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/CGStmt.cpp | 7 ++++++- test/CodeGen/asm-inout.c | 9 +++++++++ 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/lib/CodeGen/CGStmt.cpp b/lib/CodeGen/CGStmt.cpp index e65c5f468f..670167b959 100644 --- a/lib/CodeGen/CGStmt.cpp +++ b/lib/CodeGen/CGStmt.cpp @@ -1507,6 +1507,11 @@ void CodeGenFunction::EmitAsmStmt(const AsmStmt &S) { llvm::Value *Arg = EmitAsmInputLValue(S, Info, Dest, InputExpr->getType(), InOutConstraints); + if (llvm::Type* AdjTy = + getTargetHooks().adjustInlineAsmType(*this, OutputConstraint, + Arg->getType())) + Arg = Builder.CreateBitCast(Arg, AdjTy); + if (Info.allowsRegister()) InOutConstraints += llvm::utostr(i); else @@ -1565,7 +1570,7 @@ void CodeGenFunction::EmitAsmStmt(const AsmStmt &S) { } } } - if (llvm::Type* AdjTy = + if (llvm::Type* AdjTy = getTargetHooks().adjustInlineAsmType(*this, InputConstraint, Arg->getType())) Arg = Builder.CreateBitCast(Arg, AdjTy); diff --git a/test/CodeGen/asm-inout.c b/test/CodeGen/asm-inout.c index ce524fe70f..423bc4e82a 100644 --- a/test/CodeGen/asm-inout.c +++ b/test/CodeGen/asm-inout.c @@ -37,3 +37,12 @@ int test4(volatile int *addr) { return (int)oldval; // CHECK: call i8 asm "frob $0", "=r,0{{.*}}"(i8 -1) } + +// - This should have both inputs be of type x86_mmx. +// CHECK: @test5 +typedef long long __m64 __attribute__((__vector_size__(8))); +__m64 test5(__m64 __A, __m64 __B) { + // CHECK: call x86_mmx asm "pmulhuw $1, $0\0A\09", "=y,y,0,~{dirflag},~{fpsr},~{flags}"(x86_mmx %{{.}}, x86_mmx %{{.}}) + asm("pmulhuw %1, %0\n\t" : "+y" (__A) : "y" (__B)); + return __A; +} -- 2.40.0