From 0d538bbec2ebf84d615f3a3188c5f3b708646fb2 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Wed, 7 Apr 2004 04:36:53 +0000 Subject: [PATCH] Avoid emitting an extra copy on each 32-bit operation git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@12743 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/Sparc/InstSelectSimple.cpp | 15 +++++++++++---- lib/Target/Sparc/SparcV8ISelSimple.cpp | 15 +++++++++++---- lib/Target/SparcV8/InstSelectSimple.cpp | 15 +++++++++++---- lib/Target/SparcV8/SparcV8ISelSimple.cpp | 15 +++++++++++---- 4 files changed, 44 insertions(+), 16 deletions(-) diff --git a/lib/Target/Sparc/InstSelectSimple.cpp b/lib/Target/Sparc/InstSelectSimple.cpp index c845bc5d075..0d51e01ae90 100644 --- a/lib/Target/Sparc/InstSelectSimple.cpp +++ b/lib/Target/Sparc/InstSelectSimple.cpp @@ -161,6 +161,12 @@ static TypeClass getClass (const Type *T) { return cByte; } } +static TypeClass getClassB(const Type *T) { + if (T == Type::BoolTy) return cByte; + return getClass(T); +} + + /// copyConstantToRegister - Output the instructions required to put the /// specified constant into the specified register. @@ -280,9 +286,8 @@ void V8ISel::visitReturnInst(ReturnInst &I) { visitInstruction (I); return; } - } else if (I.getNumOperands () != 1) { - visitInstruction (I); } + // Just emit a 'retl' instruction to return. BuildMI(BB, V8::RETL, 0); return; @@ -293,7 +298,9 @@ void V8ISel::visitBinaryOperator (Instruction &I) { unsigned Op0Reg = getReg (I.getOperand (0)); unsigned Op1Reg = getReg (I.getOperand (1)); - unsigned ResultReg = makeAnotherReg (I.getType ()); + unsigned ResultReg = DestReg; + if (getClassB(I.getType()) != cInt) + ResultReg = makeAnotherReg (I.getType ()); unsigned OpCase = ~0; // FIXME: support long, ulong, fp. @@ -368,7 +375,7 @@ void V8ISel::visitBinaryOperator (Instruction &I) { } break; case cInt: - BuildMI (BB, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (ResultReg); + // Nothing todo here. break; default: visitInstruction (I); diff --git a/lib/Target/Sparc/SparcV8ISelSimple.cpp b/lib/Target/Sparc/SparcV8ISelSimple.cpp index c845bc5d075..0d51e01ae90 100644 --- a/lib/Target/Sparc/SparcV8ISelSimple.cpp +++ b/lib/Target/Sparc/SparcV8ISelSimple.cpp @@ -161,6 +161,12 @@ static TypeClass getClass (const Type *T) { return cByte; } } +static TypeClass getClassB(const Type *T) { + if (T == Type::BoolTy) return cByte; + return getClass(T); +} + + /// copyConstantToRegister - Output the instructions required to put the /// specified constant into the specified register. @@ -280,9 +286,8 @@ void V8ISel::visitReturnInst(ReturnInst &I) { visitInstruction (I); return; } - } else if (I.getNumOperands () != 1) { - visitInstruction (I); } + // Just emit a 'retl' instruction to return. BuildMI(BB, V8::RETL, 0); return; @@ -293,7 +298,9 @@ void V8ISel::visitBinaryOperator (Instruction &I) { unsigned Op0Reg = getReg (I.getOperand (0)); unsigned Op1Reg = getReg (I.getOperand (1)); - unsigned ResultReg = makeAnotherReg (I.getType ()); + unsigned ResultReg = DestReg; + if (getClassB(I.getType()) != cInt) + ResultReg = makeAnotherReg (I.getType ()); unsigned OpCase = ~0; // FIXME: support long, ulong, fp. @@ -368,7 +375,7 @@ void V8ISel::visitBinaryOperator (Instruction &I) { } break; case cInt: - BuildMI (BB, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (ResultReg); + // Nothing todo here. break; default: visitInstruction (I); diff --git a/lib/Target/SparcV8/InstSelectSimple.cpp b/lib/Target/SparcV8/InstSelectSimple.cpp index c845bc5d075..0d51e01ae90 100644 --- a/lib/Target/SparcV8/InstSelectSimple.cpp +++ b/lib/Target/SparcV8/InstSelectSimple.cpp @@ -161,6 +161,12 @@ static TypeClass getClass (const Type *T) { return cByte; } } +static TypeClass getClassB(const Type *T) { + if (T == Type::BoolTy) return cByte; + return getClass(T); +} + + /// copyConstantToRegister - Output the instructions required to put the /// specified constant into the specified register. @@ -280,9 +286,8 @@ void V8ISel::visitReturnInst(ReturnInst &I) { visitInstruction (I); return; } - } else if (I.getNumOperands () != 1) { - visitInstruction (I); } + // Just emit a 'retl' instruction to return. BuildMI(BB, V8::RETL, 0); return; @@ -293,7 +298,9 @@ void V8ISel::visitBinaryOperator (Instruction &I) { unsigned Op0Reg = getReg (I.getOperand (0)); unsigned Op1Reg = getReg (I.getOperand (1)); - unsigned ResultReg = makeAnotherReg (I.getType ()); + unsigned ResultReg = DestReg; + if (getClassB(I.getType()) != cInt) + ResultReg = makeAnotherReg (I.getType ()); unsigned OpCase = ~0; // FIXME: support long, ulong, fp. @@ -368,7 +375,7 @@ void V8ISel::visitBinaryOperator (Instruction &I) { } break; case cInt: - BuildMI (BB, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (ResultReg); + // Nothing todo here. break; default: visitInstruction (I); diff --git a/lib/Target/SparcV8/SparcV8ISelSimple.cpp b/lib/Target/SparcV8/SparcV8ISelSimple.cpp index c845bc5d075..0d51e01ae90 100644 --- a/lib/Target/SparcV8/SparcV8ISelSimple.cpp +++ b/lib/Target/SparcV8/SparcV8ISelSimple.cpp @@ -161,6 +161,12 @@ static TypeClass getClass (const Type *T) { return cByte; } } +static TypeClass getClassB(const Type *T) { + if (T == Type::BoolTy) return cByte; + return getClass(T); +} + + /// copyConstantToRegister - Output the instructions required to put the /// specified constant into the specified register. @@ -280,9 +286,8 @@ void V8ISel::visitReturnInst(ReturnInst &I) { visitInstruction (I); return; } - } else if (I.getNumOperands () != 1) { - visitInstruction (I); } + // Just emit a 'retl' instruction to return. BuildMI(BB, V8::RETL, 0); return; @@ -293,7 +298,9 @@ void V8ISel::visitBinaryOperator (Instruction &I) { unsigned Op0Reg = getReg (I.getOperand (0)); unsigned Op1Reg = getReg (I.getOperand (1)); - unsigned ResultReg = makeAnotherReg (I.getType ()); + unsigned ResultReg = DestReg; + if (getClassB(I.getType()) != cInt) + ResultReg = makeAnotherReg (I.getType ()); unsigned OpCase = ~0; // FIXME: support long, ulong, fp. @@ -368,7 +375,7 @@ void V8ISel::visitBinaryOperator (Instruction &I) { } break; case cInt: - BuildMI (BB, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (ResultReg); + // Nothing todo here. break; default: visitInstruction (I);