Make VAARG work with x86 long double (which is
authorDuncan Sands <baldrick@free.fr>
Mon, 3 Nov 2008 11:51:11 +0000 (11:51 +0000)
committerDuncan Sands <baldrick@free.fr>
Mon, 3 Nov 2008 11:51:11 +0000 (11:51 +0000)
10 bytes long, but is passed in 12/16 bytes).

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

lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
test/CodeGen/X86/2008-11-03-F80VAARG.ll [new file with mode: 0644]

index e1a809608a5022672460a5dfef374f9f2879b13a..8b607b443eed7de8b6b975028e53b3beaf3eeda0 100644 (file)
@@ -3438,9 +3438,9 @@ SDValue SelectionDAGLegalize::LegalizeOp(SDValue Op) {
       const Value *V = cast<SrcValueSDNode>(Node->getOperand(2))->getValue();
       SDValue VAList = DAG.getLoad(TLI.getPointerTy(), Tmp1, Tmp2, V, 0);
       // Increment the pointer, VAList, to the next vaarg
-      Tmp3 = DAG.getNode(ISD::ADD, TLI.getPointerTy(), VAList, 
-                         DAG.getConstant(VT.getSizeInBits()/8,
-                                         TLI.getPointerTy()));
+      Tmp3 = DAG.getNode(ISD::ADD, TLI.getPointerTy(), VAList,
+        DAG.getConstant(TLI.getTargetData()->getABITypeSize(VT.getTypeForMVT()),
+                        TLI.getPointerTy()));
       // Store the incremented VAList to the legalized pointer
       Tmp3 = DAG.getStore(VAList.getValue(1), Tmp3, Tmp2, V, 0);
       // Load the actual argument out of the pointer VAList
diff --git a/test/CodeGen/X86/2008-11-03-F80VAARG.ll b/test/CodeGen/X86/2008-11-03-F80VAARG.ll
new file mode 100644 (file)
index 0000000..bb9fbdb
--- /dev/null
@@ -0,0 +1,17 @@
+; RUN: llvm-as < %s | llc -march=x86 -o - | not grep 10
+
+declare void @llvm.va_start(i8*) nounwind
+
+declare void @llvm.va_copy(i8*, i8*) nounwind
+
+declare void @llvm.va_end(i8*) nounwind
+
+define x86_fp80 @test(...) nounwind {
+       %ap = alloca i8*                ; <i8**> [#uses=3]
+       %v1 = bitcast i8** %ap to i8*           ; <i8*> [#uses=1]
+       call void @llvm.va_start(i8* %v1)
+       %t1 = va_arg i8** %ap, x86_fp80         ; <x86_fp80> [#uses=1]
+       %t2 = va_arg i8** %ap, x86_fp80         ; <x86_fp80> [#uses=1]
+       %t = add x86_fp80 %t1, %t2              ; <x86_fp80> [#uses=1]
+       ret x86_fp80 %t
+}