diff options
author | Craig Topper <craig.topper@sifive.com> | 2021-01-30 12:19:14 -0800 |
---|---|---|
committer | Craig Topper <craig.topper@sifive.com> | 2021-01-30 13:16:39 -0800 |
commit | 4e04a535d8f836804d39e8861ae17d7817293c5a (patch) | |
tree | 13b16f96ecb6a2586c21966284d3c9862109d20c /llvm/utils/TableGen/CodeGenDAGPatterns.cpp | |
parent | 26d38f6d20ff137d89cb7c891b739662de1ca508 (diff) | |
download | llvm-4e04a535d8f836804d39e8861ae17d7817293c5a.zip llvm-4e04a535d8f836804d39e8861ae17d7817293c5a.tar.gz llvm-4e04a535d8f836804d39e8861ae17d7817293c5a.tar.bz2 |
[TableGen] Avoid a couple vector copies in ExpandHwModeBasedTypes.
Use vector::swap instead of copying to a local vector and clearing
the original. We can just swap into the just created local vector
instead which will move the pointers and not the data.
Use std::move in another place to avoid a copy.
Diffstat (limited to 'llvm/utils/TableGen/CodeGenDAGPatterns.cpp')
-rw-r--r-- | llvm/utils/TableGen/CodeGenDAGPatterns.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/llvm/utils/TableGen/CodeGenDAGPatterns.cpp b/llvm/utils/TableGen/CodeGenDAGPatterns.cpp index 1ca4a68..d4409db 100644 --- a/llvm/utils/TableGen/CodeGenDAGPatterns.cpp +++ b/llvm/utils/TableGen/CodeGenDAGPatterns.cpp @@ -4282,8 +4282,8 @@ static void collectModes(std::set<unsigned> &Modes, const TreePatternNode *N) { void CodeGenDAGPatterns::ExpandHwModeBasedTypes() { const CodeGenHwModes &CGH = getTargetInfo().getHwModes(); std::map<unsigned,std::vector<Predicate>> ModeChecks; - std::vector<PatternToMatch> Copy = PatternsToMatch; - PatternsToMatch.clear(); + std::vector<PatternToMatch> Copy; + PatternsToMatch.swap(Copy); auto AppendPattern = [this, &ModeChecks](PatternToMatch &P, unsigned Mode) { TreePatternNodePtr NewSrc = P.SrcPattern->clone(); @@ -4295,8 +4295,9 @@ void CodeGenDAGPatterns::ExpandHwModeBasedTypes() { std::vector<Predicate> Preds = P.Predicates; const std::vector<Predicate> &MC = ModeChecks[Mode]; llvm::append_range(Preds, MC); - PatternsToMatch.emplace_back(P.getSrcRecord(), Preds, std::move(NewSrc), - std::move(NewDst), P.getDstRegs(), + PatternsToMatch.emplace_back(P.getSrcRecord(), std::move(Preds), + std::move(NewSrc), std::move(NewDst), + P.getDstRegs(), P.getAddedComplexity(), Record::getNewUID(), Mode); }; |