[llvm] a2860b8 - [llvm-mca] Fix -Wunused-private-field after D105618

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 9 10:30:23 PDT 2021


Author: Fangrui Song
Date: 2021-07-09T10:30:16-07:00
New Revision: a2860b8b048e4a5f84586d43556d1944edbf2ace

URL: https://github.com/llvm/llvm-project/commit/a2860b8b048e4a5f84586d43556d1944edbf2ace
DIFF: https://github.com/llvm/llvm-project/commit/a2860b8b048e4a5f84586d43556d1944edbf2ace.diff

LOG: [llvm-mca] Fix -Wunused-private-field after D105618

Added: 
    

Modified: 
    llvm/tools/llvm-mca/PipelinePrinter.h
    llvm/tools/llvm-mca/llvm-mca.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/tools/llvm-mca/PipelinePrinter.h b/llvm/tools/llvm-mca/PipelinePrinter.h
index 23d55133c0cc..aadb80af86b5 100644
--- a/llvm/tools/llvm-mca/PipelinePrinter.h
+++ b/llvm/tools/llvm-mca/PipelinePrinter.h
@@ -36,11 +36,9 @@ namespace mca {
 class PipelinePrinter {
   Pipeline &P;
   llvm::SmallVector<std::unique_ptr<View>, 8> Views;
-  View::OutputKind OutputKind;
 
 public:
-  PipelinePrinter(Pipeline &pipeline, View::OutputKind OutputKind)
-      : P(pipeline), OutputKind(OutputKind) {}
+  PipelinePrinter(Pipeline &pipeline) : P(pipeline) {}
 
   void addView(std::unique_ptr<View> V) {
     P.addEventListener(V.get());

diff  --git a/llvm/tools/llvm-mca/llvm-mca.cpp b/llvm/tools/llvm-mca/llvm-mca.cpp
index 2dae2eb50b61..a4c952809302 100644
--- a/llvm/tools/llvm-mca/llvm-mca.cpp
+++ b/llvm/tools/llvm-mca/llvm-mca.cpp
@@ -579,7 +579,7 @@ int main(int argc, char **argv) {
       auto P = std::make_unique<mca::Pipeline>();
       P->appendStage(std::make_unique<mca::EntryStage>(S));
       P->appendStage(std::make_unique<mca::InstructionTables>(SM));
-      mca::PipelinePrinter Printer(*P, mca::View::OK_READABLE);
+      mca::PipelinePrinter Printer(*P);
 
       // Create the views for this pipeline, execute, and emit a report.
       if (PrintInstructionInfoView) {
@@ -614,8 +614,7 @@ int main(int argc, char **argv) {
 
     // Create a basic pipeline simulating an out-of-order backend.
     auto P = MCA.createDefaultPipeline(PO, S, *CB);
-    mca::PipelinePrinter Printer(*P, PrintJson ? mca::View::OK_JSON
-                                               : mca::View::OK_READABLE);
+    mca::PipelinePrinter Printer(*P);
 
     // When we output JSON, we add a view that contains the instructions
     // and CPU resource information.


        


More information about the llvm-commits mailing list