Skip to content

Commit

Permalink
[firtool] Move LowerLayers later in pipeline. (#7639)
Browse files Browse the repository at this point in the history
LowerLayers belongs with the other passes that lower out of FIRRTL.

Start moving it, now that first batch of passes have been fixed
to handle enough FIRRTL IR to run in presence of layer operations.
  • Loading branch information
dtzSiFive authored Sep 26, 2024
1 parent 9322802 commit 253a70b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/Firtool/Firtool.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -141,8 +141,6 @@ LogicalResult firtool::populateCHIRRTLToLowFIRRTL(mlir::PassManager &pm,
modulePM.addPass(firrtl::createLayerSinkPass());
}

pm.nest<firrtl::CircuitOp>().addPass(firrtl::createLowerLayersPass());

pm.nest<firrtl::CircuitOp>().addPass(firrtl::createInlinerPass());

// Preset the random initialization parameters for each module. The current
Expand Down Expand Up @@ -176,6 +174,8 @@ LogicalResult firtool::populateCHIRRTLToLowFIRRTL(mlir::PassManager &pm,
if (!opt.shouldDisableOptimization())
pm.nest<firrtl::CircuitOp>().addPass(firrtl::createIMConstPropPass());

pm.nest<firrtl::CircuitOp>().addPass(firrtl::createLowerLayersPass());

pm.addNestedPass<firrtl::CircuitOp>(firrtl::createAddSeqMemPortsPass());

pm.addPass(firrtl::createCreateSiFiveMetadataPass(
Expand Down

0 comments on commit 253a70b

Please sign in to comment.