RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-10-19 11:02:59 +02:00
Code Issues Projects Releases Wiki Activity

[LVI] Move ConstantRanges instead of copying.

Browse Source 
No functional change intended. Copying small (<= 64 bits) APInts isn't
expensive but bloats code by generating the slow path everywhere. Moving
doesn't care about the size of the value.

llvm-svn: 261426
This commit is contained in:
Benjamin Kramer 2016-02-20 10:40:34 +00:00
parent 56dae9ed9d
commit 50138d023d
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lib/Analysis/LazyValueInfo.cpp
View File

@ -102,7 +102,7 @@ public:
}
static LVILatticeVal getRange(ConstantRange CR) {
LVILatticeVal Res;
Res.markConstantRange(CR);
Res.markConstantRange(std::move(CR));
return Res;
}
static LVILatticeVal getOverdefined() {
@ -176,13 +176,13 @@ public:
}
/// Return true if this is a change in status.
bool markConstantRange(const ConstantRange NewR) {
bool markConstantRange(ConstantRange NewR) {
if (isConstantRange()) {
if (NewR.isEmptySet())
return markOverdefined();
bool changed = Range != NewR;
Range = NewR;
Range = std::move(NewR);
return changed;
}
@ -191,7 +191,7 @@ public:
return markOverdefined();
Tag = constantrange;
Range = NewR;
Range = std::move(NewR);
return true;
}
@ -354,7 +354,7 @@ static LVILatticeVal intersect(LVILatticeVal A, LVILatticeVal B) {
// Note: An empty range is implicitly converted to overdefined internally.
// TODO: We could instead use Undefined here since we've proven a conflict
// and thus know this path must be unreachable.
return LVILatticeVal::getRange(Range);
return LVILatticeVal::getRange(std::move(Range));
}
//===----------------------------------------------------------------------===//
@ -612,8 +612,7 @@ static LVILatticeVal getFromRangeMetadata(Instruction *BBI) {
case Instruction::Invoke:
if (MDNode *Ranges = BBI->getMetadata(LLVMContext::MD_range))
if (isa<IntegerType>(BBI->getType())) {
ConstantRange Result = getConstantRangeFromMetadata(*Ranges);
return LVILatticeVal::getRange(Result);
return LVILatticeVal::getRange(getConstantRangeFromMetadata(*Ranges));
}
break;
};
@ -1054,7 +1053,7 @@ bool getValueFromFromCondition(Value *Val, ICmpInst *ICI,
// If we're interested in the false dest, invert the condition.
if (!isTrueDest) TrueValues = TrueValues.inverse();
Result = LVILatticeVal::getRange(TrueValues);
Result = LVILatticeVal::getRange(std::move(TrueValues));
return true;
}
}
@ -1114,7 +1113,7 @@ static bool getEdgeValueLocal(Value *Val, BasicBlock *BBFrom,
} else if (i.getCaseSuccessor() == BBTo)
EdgesVals = EdgesVals.unionWith(EdgeVal);
}
Result = LVILatticeVal::getRange(EdgesVals);
Result = LVILatticeVal::getRange(std::move(EdgesVals));
return true;
}
return false;