[DebugCounter] Move the semicolon out of the DEBUG_COUNTER macro and require it to be placed at the end of each use.

This make it consistent with STATISTIC which it will often appears near. While there move one DEBUG_COUNTER instance out of an anonymous namespace. It's already declaring a static variable so the namespace is unnecessary. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@310637 91177308-0d34-0410-b5e6-96231b3b80d8

Author: topperc

docs/ProgrammersManual.rst

@@ -1224,7 +1224,7 @@ Define your DebugCounter like this:
 .. code-block:: c++
 
  DEBUG_COUNTER(DeleteAnInstruction, "passname-delete-instruction",
-"Controls which instructions get delete")
+"Controls which instructions get delete");
 
 The ``DEBUG_COUNTER`` macro defines a static variable, whose name
 is specified by the first argument. The name of the counter

include/llvm/Support/DebugCounter.h

@@ -159,7 +159,7 @@ class DebugCounter {
 
 #define DEBUG_COUNTER(VARNAME, COUNTERNAME, DESC) \
  static const unsigned VARNAME = \
- DebugCounter::registerCounter(COUNTERNAME, DESC);
+ DebugCounter::registerCounter(COUNTERNAME, DESC)
 
 } // namespace llvm
 #endif

lib/Transforms/Scalar/NewGVN.cpp

@@ -118,9 +118,9 @@ STATISTIC(NumGVNPHIOfOpsCreated, "Number of PHI of ops created");
 STATISTIC(NumGVNPHIOfOpsEliminations,
  "Number of things eliminated using PHI of ops");
 DEBUG_COUNTER(VNCounter, "newgvn-vn",
- "Controls which instructions are value numbered")
+ "Controls which instructions are value numbered");
 DEBUG_COUNTER(PHIOfOpsCounter, "newgvn-phi",
- "Controls which instructions we create phi of ops for")
+ "Controls which instructions we create phi of ops for");
 // Currently store defining access refinement is too slow due to basicaa being
 // egregiously slow. This flag lets us keep it working while we work on this
 // issue.

lib/Transforms/Utils/PredicateInfo.cpp

@@ -49,9 +49,10 @@ INITIALIZE_PASS_END(PredicateInfoPrinterLegacyPass, "print-predicateinfo",
 static cl::opt<bool> VerifyPredicateInfo(
  "verify-predicateinfo", cl::init(false), cl::Hidden,
  cl::desc("Verify PredicateInfo in legacy printer pass."));
-namespace {
 DEBUG_COUNTER(RenameCounter, "predicateinfo-rename",
- "Controls which variables are renamed with predicateinfo")
+ "Controls which variables are renamed with predicateinfo");
+
+namespace {
 // Given a predicate info that is a type of branching terminator, get the
 // branching block.
 const BasicBlock *getBranchBlock(const PredicateBase *PB) {