[llvm-commits] [llvm] r169619 - in /llvm/trunk: lib/MC/MCAssembler.cpp test/MC/ELF/no-fixup.s

[Eli Bendersky]

Author: eliben
Date: Fri Dec  7 11:59:21 2012
New Revision: 169619

URL: http://llvm.org/viewvc/llvm-project?rev=169619&view=rev
Log:
Add separate statistics for Data and Inst fragments emitted during relaxation.
Also fixes a test that was overly-sensitive to the exact order of statistics
emitted.

Modified:
    llvm/trunk/lib/MC/MCAssembler.cpp
    llvm/trunk/test/MC/ELF/no-fixup.s

Modified: llvm/trunk/lib/MC/MCAssembler.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCAssembler.cpp?rev=169619&r1=169618&r2=169619&view=diff
==============================================================================
--- llvm/trunk/lib/MC/MCAssembler.cpp (original)
+++ llvm/trunk/lib/MC/MCAssembler.cpp Fri Dec  7 11:59:21 2012
@@ -33,7 +33,9 @@
 
 namespace {
 namespace stats {
-STATISTIC(EmittedFragments, "Number of emitted assembler fragments");
+STATISTIC(EmittedFragments, "Number of emitted assembler fragments - total");
+STATISTIC(EmittedInstFragments, "Number of emitted assembler fragments - instruction");
+STATISTIC(EmittedDataFragments, "Number of emitted assembler fragments - data");
 STATISTIC(evaluateFixup, "Number of evaluated fixups");
 STATISTIC(FragmentLayouts, "Number of fragment layouts");
 STATISTIC(ObjectBytes, "Number of emitted object file bytes");
@@ -432,6 +434,7 @@
   }
 
   case MCFragment::FT_Data: {
+    ++stats::EmittedDataFragments;
     MCDataFragment &DF = cast<MCDataFragment>(F);
     assert(FragmentSize == DF.getContents().size() && "Invalid size!");
     OW->WriteBytes(DF.getContents().str());
@@ -456,6 +459,7 @@
   }
 
   case MCFragment::FT_Inst: {
+    ++stats::EmittedInstFragments;
     MCInstFragment &IF = cast<MCInstFragment>(F);
     OW->WriteBytes(StringRef(IF.getCode().begin(), IF.getCode().size()));
     break;

Modified: llvm/trunk/test/MC/ELF/no-fixup.s
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/ELF/no-fixup.s?rev=169619&r1=169618&r2=169619&view=diff
==============================================================================
--- llvm/trunk/test/MC/ELF/no-fixup.s (original)
+++ llvm/trunk/test/MC/ELF/no-fixup.s Fri Dec  7 11:59:21 2012
@@ -4,11 +4,8 @@
 // Test that we create no fixups for this file since "a" and "b" are in the
 // same fragment.
 
-// CHECK:      assembler - Number of assembler layout and relaxation steps
-// CHECK-NEXT: assembler - Number of emitted assembler fragments
-// CHECK-NEXT: assembler - Number of emitted object file bytes
+// CHECK:      assembler - Number of emitted object file bytes
 // CHECK-NEXT: assembler - Number of fragment layouts
-// CHECK-NEXT: mcexpr    - Number of MCExpr evaluations
 
 a:
   nop