maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Temporarily backing out r47337. It breaks a number of CBE tests. 

llvm-svn: 47385
This commit is contained in:
Evan Cheng 2008-02-20 18:32:05 +00:00
parent e7b462b329
commit e43b6054b1
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
llvm/lib/Target/CBackend/CBackend.cpp
View File

@ -1198,12 +1198,12 @@ void CWriter::writeOperandRaw(Value *Operand) {
} }
void CWriter::writeOperand(Value *Operand) { void CWriter::writeOperand(Value *Operand) {
if (isa<GlobalVariable>(Operand) || isDirectAlloca(Operand) || ByValParams.count(Operand)) if (isa<GlobalVariable>(Operand) || isDirectAlloca(Operand))
Out << "(&"; // Global variables are referenced as their addresses by llvm Out << "(&"; // Global variables are referenced as their addresses by llvm
writeOperandInternal(Operand); writeOperandInternal(Operand);
if (isa<GlobalVariable>(Operand) || isDirectAlloca(Operand) || ByValParams.count(Operand)) if (isa<GlobalVariable>(Operand) || isDirectAlloca(Operand))
Out << ')'; Out << ')';
} }
@ -2723,11 +2723,13 @@ void CWriter::visitCallInst(CallInst &I) {
// Check if the argument is expected to be passed by value. // Check if the argument is expected to be passed by value.
bool isOutByVal = PAL && PAL->paramHasAttr(ArgNo+1, ParamAttr::ByVal); bool isOutByVal = PAL && PAL->paramHasAttr(ArgNo+1, ParamAttr::ByVal);
// Check if this argument itself is passed in by reference. // Check if this argument itself is passed in by reference.
//bool isInByVal = ByValParams.count(*AI); bool isInByVal = ByValParams.count(*AI);
if (isOutByVal) if (isOutByVal && !isInByVal)
Out << "*("; Out << "*(";
else if (!isOutByVal && isInByVal)
Out << "&(";
writeOperand(*AI); writeOperand(*AI);
if (isOutByVal) if (isOutByVal ^ isInByVal)
Out << ")"; Out << ")";
PrintedArg = true; PrintedArg = true;
} }