]> granicus.if.org Git - llvm/commitdiff
[llvm-exegesis] deserializeMCInst(): bump SmallVector small size up to 16
authorRoman Lebedev <lebedev.ri@gmail.com>
Mon, 4 Feb 2019 09:12:13 +0000 (09:12 +0000)
committerRoman Lebedev <lebedev.ri@gmail.com>
Mon, 4 Feb 2019 09:12:13 +0000 (09:12 +0000)
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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@353022 91177308-0d34-0410-b5e6-96231b3b80d8

tools/llvm-exegesis/lib/BenchmarkResult.cpp

index e18fdf332ae25dd2cd8273ed92bc0d178d72b8e0..fae75125591e2f4efbe2ceddf08a5d2e6fbbfcf9 100644 (file)
@@ -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 << "'";