[MCA] Don't add a name to the default code region.

This is done in preparation for a patch that fixes PR41523.

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

diff --git a/test/tools/llvm-mca/X86/llvm-mca-markers-2.s b/test/tools/llvm-mca/X86/llvm-mca-markers-2.s
index 21d1030bd621b78250c75de56da290c32e829555..22ed29f74b007f306195311debe956b6b0e588c6 100644 (file)
--- a/test/tools/llvm-mca/X86/llvm-mca-markers-2.s
+++ b/test/tools/llvm-mca/X86/llvm-mca-markers-2.s
@@ -5,7 +5,7 @@
 # LLVM-MCA-END
   add %esi, %eax
 
-# CHECK:      [0] Code Region - Default
+# CHECK:      [0] Code Region
 
 # CHECK:      Iterations:        1
 # CHECK-NEXT: Instructions:      1

diff --git a/tools/llvm-mca/CodeRegion.cpp b/tools/llvm-mca/CodeRegion.cpp
index 6d4192250aa64aa9079386ea1e00d30948671dc7..eb04ea3f0d8ce0e71295c668181747bc58ff8a15 100644 (file)
--- a/tools/llvm-mca/CodeRegion.cpp
+++ b/tools/llvm-mca/CodeRegion.cpp
@@ -18,7 +18,7 @@ namespace mca {
 
 CodeRegions::CodeRegions(llvm::SourceMgr &S) : SM(S) {
   // Create a default region for the input code sequence.
-  Regions.emplace_back(make_unique<CodeRegion>("Default", SMLoc()));
+  Regions.emplace_back(make_unique<CodeRegion>("", SMLoc()));
 }
 
 bool CodeRegion::isLocInRange(SMLoc Loc) const {

diff --git a/tools/llvm-mca/CodeRegion.h b/tools/llvm-mca/CodeRegion.h
index cc474b78ca7ec447f73898d5ef7f1a2a78c55a71..80532e311edb4630cca034aa736e92595dc59e38 100644 (file)
--- a/tools/llvm-mca/CodeRegion.h
+++ b/tools/llvm-mca/CodeRegion.h
@@ -90,6 +90,8 @@ class CodeRegions {
   CodeRegions &operator=(const CodeRegions &) = delete;
 
 public:
+  CodeRegions(llvm::SourceMgr &S);
+
   typedef std::vector<UniqueCodeRegion>::iterator iterator;
   typedef std::vector<UniqueCodeRegion>::const_iterator const_iterator;
 
@@ -103,8 +105,6 @@ public:
   void addInstruction(const llvm::MCInst &Instruction);
   llvm::SourceMgr &getSourceMgr() const { return SM; }
 
-  CodeRegions(llvm::SourceMgr &S);
-
   llvm::ArrayRef<llvm::MCInst> getInstructionSequence(unsigned Idx) const {
     return Regions[Idx]->getInstructions();
   }

diff --git a/tools/llvm-mca/CodeRegionGenerator.cpp b/tools/llvm-mca/CodeRegionGenerator.cpp
index 4660af2c40aca37fca05172259029d6171867d05..14b470a417c873cdc10aa2512a198f3d0c4c8a40 100644 (file)
--- a/tools/llvm-mca/CodeRegionGenerator.cpp
+++ b/tools/llvm-mca/CodeRegionGenerator.cpp
@@ -115,7 +115,6 @@ Expected<const CodeRegions &> AsmCodeRegionGenerator::parseCodeRegions() {
   MCACommentConsumer CC(Regions);
   Lexer.setCommentConsumer(&CC);
 
-  // Create a target-specific parser and perform the parse.
   std::unique_ptr<MCTargetAsmParser> TAP(
       TheTarget.createMCAsmParser(STI, *Parser, MCII, Opts));
   if (!TAP)
@@ -125,7 +124,7 @@ Expected<const CodeRegions &> AsmCodeRegionGenerator::parseCodeRegions() {
   Parser->setTargetParser(*TAP);
   Parser->Run(false);
 
-  // Get the assembler dialect from the input.  llvm-mca will use this as the
+  // Set the assembler dialect from the input. llvm-mca will use this as the
   // default dialect when printing reports.
   AssemblerDialect = Parser->getAssemblerDialect();
   return Regions;