mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-04-11 00:39:36 +00:00
Recommited reworked r156804:
SelectionDAGBuilder::Clusterify : main functinality was replaced with CRSBuilder::optimize, so big part of Clusterify's code was reduced. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@157046 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
be97ae9e89
commit
05cfe2eda6
@ -51,6 +51,7 @@
|
||||
#include "llvm/Target/TargetLowering.h"
|
||||
#include "llvm/Target/TargetOptions.h"
|
||||
#include "llvm/Support/CommandLine.h"
|
||||
#include "llvm/Support/CRSBuilder.h"
|
||||
#include "llvm/Support/Debug.h"
|
||||
#include "llvm/Support/ErrorHandling.h"
|
||||
#include "llvm/Support/MathExtras.h"
|
||||
@ -2408,57 +2409,43 @@ bool SelectionDAGBuilder::handleBitTestsSwitchCase(CaseRec& CR,
|
||||
/// Clusterify - Transform simple list of Cases into list of CaseRange's
|
||||
size_t SelectionDAGBuilder::Clusterify(CaseVector& Cases,
|
||||
const SwitchInst& SI) {
|
||||
size_t numCmps = 0;
|
||||
|
||||
/// Use a shorter form of declaration, and also
|
||||
/// show the we want to use CRSBuilder as Clusterifier.
|
||||
typedef CRSBuilderBase<MachineBasicBlock, true> Clusterifier;
|
||||
|
||||
Clusterifier TheClusterifier;
|
||||
|
||||
BranchProbabilityInfo *BPI = FuncInfo.BPI;
|
||||
// Start with "simple" cases
|
||||
for (SwitchInst::ConstCaseIt i = SI.case_begin(), e = SI.case_end();
|
||||
i != e; ++i) {
|
||||
const BasicBlock *SuccBB = i.getCaseSuccessor();
|
||||
MachineBasicBlock *SMBB = FuncInfo.MBBMap[SuccBB];
|
||||
|
||||
uint32_t ExtraWeight = BPI ? BPI->getEdgeWeight(SI.getParent(), SuccBB) : 0;
|
||||
|
||||
Cases.push_back(Case(i.getCaseValue(), i.getCaseValue(),
|
||||
SMBB, ExtraWeight));
|
||||
TheClusterifier.add(i.getCaseValueEx(), SMBB);
|
||||
}
|
||||
std::sort(Cases.begin(), Cases.end(), CaseCmp());
|
||||
|
||||
// Merge case into clusters
|
||||
if (Cases.size() >= 2)
|
||||
// Must recompute end() each iteration because it may be
|
||||
// invalidated by erase if we hold on to it
|
||||
for (CaseItr I = Cases.begin(), J = llvm::next(Cases.begin());
|
||||
J != Cases.end(); ) {
|
||||
const APInt& nextValue = cast<ConstantInt>(J->Low)->getValue();
|
||||
const APInt& currentValue = cast<ConstantInt>(I->High)->getValue();
|
||||
MachineBasicBlock* nextBB = J->BB;
|
||||
MachineBasicBlock* currentBB = I->BB;
|
||||
|
||||
// If the two neighboring cases go to the same destination, merge them
|
||||
// into a single case.
|
||||
if ((nextValue - currentValue == 1) && (currentBB == nextBB)) {
|
||||
I->High = J->High;
|
||||
J = Cases.erase(J);
|
||||
|
||||
if (BranchProbabilityInfo *BPI = FuncInfo.BPI) {
|
||||
uint32_t CurWeight = currentBB->getBasicBlock() ?
|
||||
BPI->getEdgeWeight(SI.getParent(), currentBB->getBasicBlock()) : 16;
|
||||
uint32_t NextWeight = nextBB->getBasicBlock() ?
|
||||
BPI->getEdgeWeight(SI.getParent(), nextBB->getBasicBlock()) : 16;
|
||||
|
||||
BPI->setEdgeWeight(SI.getParent(), currentBB->getBasicBlock(),
|
||||
CurWeight + NextWeight);
|
||||
}
|
||||
} else {
|
||||
I = J++;
|
||||
}
|
||||
|
||||
TheClusterifier.optimize();
|
||||
|
||||
BranchProbabilityInfo *BPI = FuncInfo.BPI;
|
||||
size_t numCmps = 0;
|
||||
for (Clusterifier::RangeIterator i = TheClusterifier.begin(),
|
||||
e = TheClusterifier.end(); i != e; ++i, ++numCmps) {
|
||||
Clusterifier::Cluster &C = *i;
|
||||
unsigned W = 0;
|
||||
if (BPI) {
|
||||
W = BPI->getEdgeWeight(SI.getParent(), C.second->getBasicBlock());
|
||||
if (!W)
|
||||
W = 16;
|
||||
W *= C.first.Weight;
|
||||
BPI->setEdgeWeight(SI.getParent(), C.second->getBasicBlock(), W);
|
||||
}
|
||||
|
||||
for (CaseItr I=Cases.begin(), E=Cases.end(); I!=E; ++I, ++numCmps) {
|
||||
if (I->Low != I->High)
|
||||
// A range counts double, since it requires two compares.
|
||||
++numCmps;
|
||||
Cases.push_back(Case(C.first.Low, C.first.High, C.second, W));
|
||||
|
||||
if (C.first.Low != C.first.High)
|
||||
// A range counts double, since it requires two compares.
|
||||
++numCmps;
|
||||
}
|
||||
|
||||
return numCmps;
|
||||
|
@ -180,17 +180,6 @@ private:
|
||||
|
||||
typedef std::vector<CaseRec> CaseRecVector;
|
||||
|
||||
/// The comparison function for sorting the switch case values in the vector.
|
||||
/// WARNING: Case ranges should be disjoint!
|
||||
struct CaseCmp {
|
||||
bool operator()(const Case &C1, const Case &C2) {
|
||||
assert(isa<ConstantInt>(C1.Low) && isa<ConstantInt>(C2.High));
|
||||
const ConstantInt* CI1 = cast<const ConstantInt>(C1.Low);
|
||||
const ConstantInt* CI2 = cast<const ConstantInt>(C2.High);
|
||||
return CI1->getValue().ult(CI2->getValue());
|
||||
}
|
||||
};
|
||||
|
||||
struct CaseBitsCmp {
|
||||
bool operator()(const CaseBits &C1, const CaseBits &C2) {
|
||||
return C1.Bits > C2.Bits;
|
||||
|
Loading…
x
Reference in New Issue
Block a user