[PATCH] D52565: [llvm-exegesis] Remove unused headers and fix naming issues

[Fangrui Song via Phabricator via llvm-commits]

MaskRay created this revision.
MaskRay added a reviewer: courbet.
Herald added subscribers: llvm-commits, tschuett.

Repository:
  rL LLVM

https://reviews.llvm.org/D52565

Files:
  tools/llvm-exegesis/llvm-exegesis.cpp


Index: tools/llvm-exegesis/llvm-exegesis.cpp
===================================================================
--- tools/llvm-exegesis/llvm-exegesis.cpp
+++ tools/llvm-exegesis/llvm-exegesis.cpp
@@ -36,9 +36,7 @@
 #include "llvm/Support/TargetRegistry.h"
 #include "llvm/Support/TargetSelect.h"
 #include <algorithm>
-#include <random>
 #include <string>
-#include <unordered_map>
 
 static llvm::cl::opt<unsigned>
     OpcodeIndex("opcode-index", llvm::cl::desc("opcode to measure, by index"),
@@ -160,10 +158,10 @@
 
   // Implementation of the llvm::MCStreamer interface. We only care about
   // instructions.
-  void EmitInstruction(const llvm::MCInst &instruction,
-                       const llvm::MCSubtargetInfo &mc_subtarget_info,
+  void EmitInstruction(const llvm::MCInst &Instruction,
+                       const llvm::MCSubtargetInfo &STI,
                        bool PrintSchedInfo) override {
-    Result->Instructions.push_back(instruction);
+    Result->Instructions.push_back(Instruction);
   }
 
   // Implementation of the llvm::AsmCommentConsumer.
@@ -211,17 +209,17 @@
 
 private:
   // We only care about instructions, we don't implement this part of the API.
-  void EmitCommonSymbol(llvm::MCSymbol *symbol, uint64_t size,
-                        unsigned byte_alignment) override {}
-  bool EmitSymbolAttribute(llvm::MCSymbol *symbol,
-                           llvm::MCSymbolAttr attribute) override {
+  void EmitCommonSymbol(llvm::MCSymbol *Symbol, uint64_t Size,
+                        unsigned ByteAlignment) override {}
+  bool EmitSymbolAttribute(llvm::MCSymbol *Symbol,
+                           llvm::MCSymbolAttr Attribute) override {
     return false;
   }
-  void EmitValueToAlignment(unsigned byte_alignment, int64_t value,
-                            unsigned value_size,
-                            unsigned max_bytes_to_emit) override {}
-  void EmitZerofill(llvm::MCSection *section, llvm::MCSymbol *symbol,
-                    uint64_t size, unsigned byte_alignment,
+  void EmitValueToAlignment(unsigned ByteAlignment, int64_t Value,
+                            unsigned ValueSize,
+                            unsigned MaxBytesToEmit) override {}
+  void EmitZerofill(llvm::MCSection *Section, llvm::MCSymbol *Symbol,
+                    uint64_t Size, unsigned ByteAlignment,
                     llvm::SMLoc Loc) override {}
 
   unsigned findRegisterByName(const llvm::StringRef RegName) const {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52565.167167.patch
Type: text/x-patch
Size: 2464 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180926/f6c4a2a7/attachment.bin>