[Support] No ";" in CIRCT_DEBUG_SCOPED_PASS_LOGGER

Drop the trailing semicolon after the `CIRCT_DEBUG_SCOPED_PASS_LOGGER`.
This better aligns with other debug macros.

h/t @rwy7 for the suggestion.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>

Author: seldridge

include/circt/Support/Debug.h

@@ -68,11 +68,11 @@ class ScopedDebugPassLogger {
 };
 
 #define CIRCT_DEBUG_SCOPED_PASS_LOGGER(PASS)                                   \
-  ScopedDebugPassLogger _scopedDebugPassLogger(PASS);
+  ScopedDebugPassLogger _scopedDebugPassLogger(PASS)
 #else
 #define CIRCT_DEBUG_SCOPED_PASS_LOGGER(PASS)                                   \
   do {                                                                         \
-  } while (0);
+  } while (0)
 #endif
 
 } // namespace circt

lib/Dialect/FIRRTL/Transforms/AssignOutputDirs.cpp

@@ -84,7 +84,7 @@ struct AssignOutputDirsPass
 } // namespace
 
 void AssignOutputDirsPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
   SmallString<64> outputDir(outputDirOption);
   if (fs::make_absolute(outputDir)) {
     emitError(mlir::UnknownLoc::get(&getContext()),

lib/Dialect/FIRRTL/Transforms/BlackBoxReader.cpp

@@ -122,7 +122,7 @@ struct BlackBoxReaderPass
 
 /// Emit the annotated source code for black boxes in a circuit.
 void BlackBoxReaderPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
   CircuitOp circuitOp = getOperation();
   CircuitNamespace ns(circuitOp);
 

lib/Dialect/FIRRTL/Transforms/EliminateWires.cpp

@@ -47,7 +47,7 @@ struct EliminateWiresPass
 } // end anonymous namespace
 
 void EliminateWiresPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   auto module = getOperation();
   auto &dominance = getAnalysis<mlir::DominanceInfo>();

lib/Dialect/FIRRTL/Transforms/GrandCentral.cpp

@@ -1809,7 +1809,7 @@ GrandCentralPass::getEnclosingModule(Value value, FlatSymbolRefAttr sym) {
 
 /// This method contains the business logic of this pass.
 void GrandCentralPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   CircuitOp circuitOp = getOperation();
 

lib/Dialect/FIRRTL/Transforms/IMDeadCodeElim.cpp

@@ -313,7 +313,7 @@ void IMDeadCodeElimPass::forwardConstantOutputPort(FModuleOp module) {
 }
 
 void IMDeadCodeElimPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   auto circuits = getOperation().getOps<CircuitOp>();
   if (circuits.empty())

lib/Dialect/FIRRTL/Transforms/InjectDUTHierarchy.cpp

@@ -131,7 +131,7 @@ static void addHierarchy(hw::HierPathOp path, FModuleOp dut,
 }
 
 void InjectDUTHierarchy::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   CircuitOp circuit = getOperation();
 

lib/Dialect/FIRRTL/Transforms/LayerSink.cpp

@@ -478,7 +478,7 @@ struct LayerSinkPass final
 } // namespace
 
 void LayerSinkPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   auto circuit = getOperation();
   LLVM_DEBUG(llvm::dbgs() << "Circuit: '" << circuit.getName() << "'\n");

lib/Dialect/FIRRTL/Transforms/LowerAnnotations.cpp

@@ -1210,7 +1210,7 @@ LogicalResult LowerAnnotationsPass::solveWiringProblems(ApplyState &state) {
 
 // This is the main entrypoint for the lowering pass.
 void LowerAnnotationsPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   CircuitOp circuit = getOperation();
   SymbolTable modules(circuit);

lib/Dialect/FIRRTL/Transforms/LowerOpenAggs.cpp

@@ -835,7 +835,7 @@ struct LowerOpenAggsPass
 
 // This is the main entrypoint for the lowering pass.
 void LowerOpenAggsPass::runOnOperation() {
-  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this)
+  CIRCT_DEBUG_SCOPED_PASS_LOGGER(this);
 
   SmallVector<Operation *, 0> ops(getOperation().getOps<FModuleLike>());