[mips] Do not use function CC_MipsN_VarArg unless the function being analyzed
authorAkira Hatanaka <ahatanaka@mips.com>
Tue, 5 Feb 2013 21:18:11 +0000 (21:18 +0000)
committerAkira Hatanaka <ahatanaka@mips.com>
Tue, 5 Feb 2013 21:18:11 +0000 (21:18 +0000)
is a vararg function.

The original code was examining flag OutputArg::IsFixed to determine whether
CC_MipsN_VarArg or CC_MipsN should be called. This is not correct, since this
flag is often set to false when the function being analyzed is a non-variadic
function.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@174442 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/Mips/MipsISelLowering.cpp
test/CodeGen/Mips/mips64-expand-libcall.ll [new file with mode: 0644]

index ff0064e68c81ea25934bce8f5194e5eb53aefc7f..2ff369cfab2832cec5fc10bb2a1913e87f71876c 100644 (file)
@@ -3793,7 +3793,7 @@ MipsTargetLowering::MipsCC::MipsCC(CallingConv::ID CallConv, bool IsVarArg,
     IntArgRegs = Mips64IntRegs;
     ShadowRegs = Mips64DPRegs;
     FixedFn = CC_MipsN;
-    VarFn = CC_MipsN_VarArg;
+    VarFn = IsVarArg ? CC_MipsN_VarArg : CC_MipsN;
   }
 
   if (CallConv == CallingConv::Fast) {
diff --git a/test/CodeGen/Mips/mips64-expand-libcall.ll b/test/CodeGen/Mips/mips64-expand-libcall.ll
new file mode 100644 (file)
index 0000000..4a67211
--- /dev/null
@@ -0,0 +1,14 @@
+; RUN: llc -march=mips64el -mcpu=mips64r2 -O3 < %s | FileCheck %s
+
+; Check that %add is not passed in an integer register.
+;
+; CHECK-NOT: dmfc1 $4
+
+define double @callfloor(double %d) nounwind readnone {
+entry:
+  %add = fadd double %d, 1.000000e+00
+  %call = tail call double @floor(double %add) nounwind readnone
+  ret double %call
+}
+
+declare double @floor(double) nounwind readnone