[llvm] r353022 - [llvm-exegesis] deserializeMCInst(): bump SmallVector small size up to 16
Roman Lebedev via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 4 01:12:13 PST 2019
Author: lebedevri
Date: Mon Feb 4 01:12:13 2019
New Revision: 353022
URL: http://llvm.org/viewvc/llvm-project?rev=353022&view=rev
Log:
[llvm-exegesis] deserializeMCInst(): bump SmallVector small size up to 16
Summary:
... from 8.
`VALIGNDZ128rmbik XMM0 XMM0 K1 XMM3 RDI i_0x1 i_0x0 i_0x1` instruction already has 9 components.
It does not matter much in terms of performance, but avoiding allocation seems to come with low cost here..
Reviewers: courbet, gchatelet
Reviewed By: courbet
Subscribers: tschuett, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D57654
Modified:
llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp
Modified: llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp?rev=353022&r1=353021&r2=353022&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp (original)
+++ llvm/trunk/tools/llvm-exegesis/lib/BenchmarkResult.cpp Mon Feb 4 01:12:13 2019
@@ -40,7 +40,7 @@ struct YamlContext {
}
void deserializeMCInst(llvm::StringRef String, llvm::MCInst &Value) {
- llvm::SmallVector<llvm::StringRef, 8> Pieces;
+ llvm::SmallVector<llvm::StringRef, 16> Pieces;
String.split(Pieces, " ", /* MaxSplit */ -1, /* KeepEmpty */ false);
if (Pieces.empty()) {
ErrorStream << "Unknown Instruction: '" << String << "'";
More information about the llvm-commits
mailing list