[llvm] r328184 - [TableGen] Use empty emplace_back to add defaulted constructed objects to vectors to avoid using resize(size()+1). NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 21 23:15:10 PDT 2018
Author: ctopper
Date: Wed Mar 21 23:15:10 2018
New Revision: 328184
URL: http://llvm.org/viewvc/llvm-project?rev=328184&view=rev
Log:
[TableGen] Use empty emplace_back to add defaulted constructed objects to vectors to avoid using resize(size()+1). NFC
Modified:
llvm/trunk/utils/TableGen/CodeGenSchedule.cpp
Modified: llvm/trunk/utils/TableGen/CodeGenSchedule.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeGenSchedule.cpp?rev=328184&r1=328183&r2=328184&view=diff
==============================================================================
--- llvm/trunk/utils/TableGen/CodeGenSchedule.cpp (original)
+++ llvm/trunk/utils/TableGen/CodeGenSchedule.cpp Wed Mar 21 23:15:10 2018
@@ -1312,7 +1312,7 @@ void PredTransitions::substituteVariants
// Build up a set of partial results starting at the back of
// PredTransitions. Remember the first new transition.
unsigned StartIdx = TransVec.size();
- TransVec.resize(TransVec.size() + 1);
+ TransVec.emplace_back();
TransVec.back().PredTerm = Trans.PredTerm;
TransVec.back().ProcIndices = Trans.ProcIndices;
@@ -1323,7 +1323,7 @@ void PredTransitions::substituteVariants
// Push a new (empty) write sequence onto all partial Transitions.
for (std::vector<PredTransition>::iterator I =
TransVec.begin() + StartIdx, E = TransVec.end(); I != E; ++I) {
- I->WriteSequences.resize(I->WriteSequences.size() + 1);
+ I->WriteSequences.emplace_back();
}
substituteVariantOperand(*WSI, /*IsRead=*/false, StartIdx);
}
@@ -1334,7 +1334,7 @@ void PredTransitions::substituteVariants
// Push a new (empty) read sequence onto all partial Transitions.
for (std::vector<PredTransition>::iterator I =
TransVec.begin() + StartIdx, E = TransVec.end(); I != E; ++I) {
- I->ReadSequences.resize(I->ReadSequences.size() + 1);
+ I->ReadSequences.emplace_back();
}
substituteVariantOperand(*RSI, /*IsRead=*/true, StartIdx);
}
@@ -1388,16 +1388,15 @@ void CodeGenSchedModels::inferFromRW(Arr
// Create a seed transition with an empty PredTerm and the expanded sequences
// of SchedWrites for the current SchedClass.
std::vector<PredTransition> LastTransitions;
- LastTransitions.resize(1);
+ LastTransitions.emplace_back();
LastTransitions.back().ProcIndices.append(ProcIndices.begin(),
ProcIndices.end());
for (unsigned WriteIdx : OperWrites) {
IdxVec WriteSeq;
expandRWSequence(WriteIdx, WriteSeq, /*IsRead=*/false);
- unsigned Idx = LastTransitions[0].WriteSequences.size();
- LastTransitions[0].WriteSequences.resize(Idx + 1);
- SmallVectorImpl<unsigned> &Seq = LastTransitions[0].WriteSequences[Idx];
+ LastTransitions[0].WriteSequences.emplace_back();
+ SmallVectorImpl<unsigned> &Seq = LastTransitions[0].WriteSequences.back();
Seq.append(WriteSeq.begin(), WriteSeq.end());
DEBUG(dbgs() << "("; dumpIdxVec(Seq); dbgs() << ") ");
}
@@ -1405,9 +1404,8 @@ void CodeGenSchedModels::inferFromRW(Arr
for (unsigned ReadIdx : OperReads) {
IdxVec ReadSeq;
expandRWSequence(ReadIdx, ReadSeq, /*IsRead=*/true);
- unsigned Idx = LastTransitions[0].ReadSequences.size();
- LastTransitions[0].ReadSequences.resize(Idx + 1);
- SmallVectorImpl<unsigned> &Seq = LastTransitions[0].ReadSequences[Idx];
+ LastTransitions[0].ReadSequences.emplace_back();
+ SmallVectorImpl<unsigned> &Seq = LastTransitions[0].ReadSequences.back();
Seq.append(ReadSeq.begin(), ReadSeq.end());
DEBUG(dbgs() << "("; dumpIdxVec(Seq); dbgs() << ") ");
}
More information about the llvm-commits
mailing list