From 5144ed1e35fe55f2e5171e359ee733583a21f641 Mon Sep 17 00:00:00 2001 From: Evan Cheng Date: Wed, 20 Feb 2008 18:32:05 +0000 Subject: Temporarily backing out r47337. It breaks a number of CBE tests. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47385 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/CBackend/CBackend.cpp | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'lib') diff --git a/lib/Target/CBackend/CBackend.cpp b/lib/Target/CBackend/CBackend.cpp index 305c8de..91ca97e 100644 --- a/lib/Target/CBackend/CBackend.cpp +++ b/lib/Target/CBackend/CBackend.cpp @@ -1198,12 +1198,12 @@ void CWriter::writeOperandRaw(Value *Operand) { } void CWriter::writeOperand(Value *Operand) { - if (isa(Operand) || isDirectAlloca(Operand) || ByValParams.count(Operand)) + if (isa(Operand) || isDirectAlloca(Operand)) Out << "(&"; // Global variables are referenced as their addresses by llvm writeOperandInternal(Operand); - if (isa(Operand) || isDirectAlloca(Operand) || ByValParams.count(Operand)) + if (isa(Operand) || isDirectAlloca(Operand)) Out << ')'; } @@ -2723,11 +2723,13 @@ void CWriter::visitCallInst(CallInst &I) { // Check if the argument is expected to be passed by value. bool isOutByVal = PAL && PAL->paramHasAttr(ArgNo+1, ParamAttr::ByVal); // Check if this argument itself is passed in by reference. - //bool isInByVal = ByValParams.count(*AI); - if (isOutByVal) + bool isInByVal = ByValParams.count(*AI); + if (isOutByVal && !isInByVal) Out << "*("; + else if (!isOutByVal && isInByVal) + Out << "&("; writeOperand(*AI); - if (isOutByVal) + if (isOutByVal ^ isInByVal) Out << ")"; PrintedArg = true; } -- cgit v1.1