diff options
| author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2013-04-02 22:27:45 +0000 |
|---|---|---|
| committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2013-04-02 22:27:45 +0000 |
| commit | 2ccdbc6675fd5bafb6da65dfca850b6e2587f9f3 (patch) | |
| tree | a5aea03f67131d17c0a559484051bb4fd4afdd78 /lib/CodeGen | |
| parent | d0a0757c3b4bf4277267c7bb869651ef01493060 (diff) | |
| download | external_llvm-2ccdbc6675fd5bafb6da65dfca850b6e2587f9f3.zip external_llvm-2ccdbc6675fd5bafb6da65dfca850b6e2587f9f3.tar.gz external_llvm-2ccdbc6675fd5bafb6da65dfca850b6e2587f9f3.tar.bz2 | |
Allow MachineTraceMetrics to be used when the model has no resources.
It it still possible to extract information from itineraries, for
example.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@178582 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen')
| -rw-r--r-- | lib/CodeGen/EarlyIfConversion.cpp | 4 | ||||
| -rw-r--r-- | lib/CodeGen/MachineTraceMetrics.cpp | 14 |
2 files changed, 11 insertions, 7 deletions
diff --git a/lib/CodeGen/EarlyIfConversion.cpp b/lib/CodeGen/EarlyIfConversion.cpp index fc76736..5447df0 100644 --- a/lib/CodeGen/EarlyIfConversion.cpp +++ b/lib/CodeGen/EarlyIfConversion.cpp @@ -677,10 +677,6 @@ bool EarlyIfConverter::shouldConvertIf() { if (Stress) return true; - // Without a scheduling model, we can't make decisions. - if (!SchedModel->hasInstrSchedModel()) - return false; - if (!MinInstr) MinInstr = Traces->getEnsemble(MachineTraceMetrics::TS_MinInstrCount); diff --git a/lib/CodeGen/MachineTraceMetrics.cpp b/lib/CodeGen/MachineTraceMetrics.cpp index c154b5c..49d8c4e 100644 --- a/lib/CodeGen/MachineTraceMetrics.cpp +++ b/lib/CodeGen/MachineTraceMetrics.cpp @@ -105,6 +105,8 @@ MachineTraceMetrics::getResources(const MachineBasicBlock *MBB) { FBI->HasCalls = true; // Count processor resources used. + if (!SchedModel.hasInstrSchedModel()) + continue; const MCSchedClassDesc *SC = SchedModel.resolveSchedClass(MI); if (!SC->isValid()) continue; @@ -132,7 +134,9 @@ MachineTraceMetrics::getProcResourceCycles(unsigned MBBNum) const { assert(BlockInfo[MBBNum].hasResources() && "getResources() must be called before getProcResourceCycles()"); unsigned PRKinds = SchedModel.getNumProcResourceKinds(); - return ArrayRef<unsigned>(&ProcResourceCycles[MBBNum * PRKinds], PRKinds); + assert((MBBNum+1) * PRKinds <= ProcResourceCycles.size()); + return ArrayRef<unsigned>(ProcResourceCycles.data() + MBBNum * PRKinds, + PRKinds); } @@ -251,7 +255,9 @@ ArrayRef<unsigned> MachineTraceMetrics::Ensemble:: getProcResourceDepths(unsigned MBBNum) const { unsigned PRKinds = MTM.SchedModel.getNumProcResourceKinds(); - return ArrayRef<unsigned>(&ProcResourceDepths[MBBNum * PRKinds], PRKinds); + assert((MBBNum+1) * PRKinds <= ProcResourceDepths.size()); + return ArrayRef<unsigned>(ProcResourceDepths.data() + MBBNum * PRKinds, + PRKinds); } /// Get an array of processor resource heights for MBB. Indexed by processor @@ -263,7 +269,9 @@ ArrayRef<unsigned> MachineTraceMetrics::Ensemble:: getProcResourceHeights(unsigned MBBNum) const { unsigned PRKinds = MTM.SchedModel.getNumProcResourceKinds(); - return ArrayRef<unsigned>(&ProcResourceHeights[MBBNum * PRKinds], PRKinds); + assert((MBBNum+1) * PRKinds <= ProcResourceHeights.size()); + return ArrayRef<unsigned>(ProcResourceHeights.data() + MBBNum * PRKinds, + PRKinds); } //===----------------------------------------------------------------------===// |