fixup! [FIRRTL] Make IMDCE work for ops w/ regions/blocks

llvm · Nov 26, 2024 · c4d490a · c4d490a
1 parent 08c3c8a
commit c4d490a
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/lib/Dialect/FIRRTL/Transforms/IMDeadCodeElim.cpp b/lib/Dialect/FIRRTL/Transforms/IMDeadCodeElim.cpp
@@ -263,14 +263,14 @@ void IMDeadCodeElimPass::markBlockExecutable(Block *block) {
     else if (isa<FConnectLike>(op))
       // Skip connect op.
       continue;
-    else if (hasUnknownSideEffect(&op))
+    else if (hasUnknownSideEffect(&op)) {
       markUnknownSideEffectOp(&op);
-
-    // Recursively mark any blocks contained within these operations as
-    // executable.
-    for (auto &region : op.getRegions())
-      for (auto &block : region.getBlocks())
-        markBlockExecutable(&block);
+      // Recursively mark any blocks contained within these operations as
+      // executable.
+      for (auto &region : op.getRegions())
+        for (auto &block : region.getBlocks())
+          markBlockExecutable(&block);
+    }
 
     // TODO: Handle attach etc.
   }