diff --git a/passes/pmgen/README.md b/passes/pmgen/README.md index 542c2c0e8..15b4f79a1 100644 --- a/passes/pmgen/README.md +++ b/passes/pmgen/README.md @@ -34,8 +34,8 @@ for the pattern`` and calls the callback function for each found match: pm.run_foobar([&](){ - log("found matching 'foo' cell: %s\n", log_id(pm.st.foo)); - log(" with 'bar' cell: %s\n", log_id(pm.st.bar)); + log("found matching 'foo' cell: %s\n", pm.st.foo); + log(" with 'bar' cell: %s\n", pm.st.bar); }); The `.pmg` file declares matcher state variables that are accessible via the diff --git a/techlibs/ice40/ice40_dsp.cc b/techlibs/ice40/ice40_dsp.cc index 7942943c4..1d0f98f2f 100644 --- a/techlibs/ice40/ice40_dsp.cc +++ b/techlibs/ice40/ice40_dsp.cc @@ -31,15 +31,15 @@ void create_ice40_dsp(ice40_dsp_pm &pm) log("Checking %s.%s for iCE40 DSP inference.\n", pm.module, st.mul); - log_debug("ffA: %s\n", log_id(st.ffA, "--")); - log_debug("ffB: %s\n", log_id(st.ffB, "--")); - log_debug("ffCD: %s\n", log_id(st.ffCD, "--")); - log_debug("mul: %s\n", log_id(st.mul, "--")); - log_debug("ffFJKG: %s\n", log_id(st.ffFJKG, "--")); - log_debug("ffH: %s\n", log_id(st.ffH, "--")); - log_debug("add: %s\n", log_id(st.add, "--")); - log_debug("mux: %s\n", log_id(st.mux, "--")); - log_debug("ffO: %s\n", log_id(st.ffO, "--")); + log_debug("ffA: %s\n", st.ffA ? st.ffA->name.unescape() : "--"); + log_debug("ffB: %s\n", st.ffB ? st.ffB->name.unescape() : "--"); + log_debug("ffCD: %s\n", st.ffCD ? st.ffCD->name.unescape() : "--"); + log_debug("mul: %s\n", st.mul ? st.mul->name.unescape() : "--"); + log_debug("ffFJKG: %s\n", st.ffFJKG ? st.ffFJKG->name.unescape() : "--"); + log_debug("ffH: %s\n", st.ffH ? st.ffH->name.unescape() : "--"); + log_debug("add: %s\n", st.add ? st.add->name.unescape() : "--"); + log_debug("mux: %s\n", st.mux ? st.mux->name.unescape() : "--"); + log_debug("ffO: %s\n", st.ffO ? st.ffO->name.unescape() : "--"); log_debug("\n"); if (GetSize(st.sigA) > 16) { diff --git a/techlibs/ice40/ice40_wrapcarry.cc b/techlibs/ice40/ice40_wrapcarry.cc index f62019617..63ebdbfcf 100644 --- a/techlibs/ice40/ice40_wrapcarry.cc +++ b/techlibs/ice40/ice40_wrapcarry.cc @@ -31,8 +31,8 @@ void create_ice40_wrapcarry(ice40_wrapcarry_pm &pm) #if 0 log("\n"); - log("carry: %s\n", log_id(st.carry, "--")); - log("lut: %s\n", log_id(st.lut, "--")); + log("carry: %s\n", st.carry ? st.carry->name.unescape() : "--"); + log("lut: %s\n", st.lut ? st.lut->name.unescape() : "--"); #endif log(" replacing SB_LUT + SB_CARRY with $__ICE40_CARRY_WRAPPER cell.\n"); diff --git a/techlibs/microchip/microchip_dsp.cc b/techlibs/microchip/microchip_dsp.cc index 01c77644f..ff86049eb 100644 --- a/techlibs/microchip/microchip_dsp.cc +++ b/techlibs/microchip/microchip_dsp.cc @@ -195,7 +195,7 @@ void microchip_dsp_packC(microchip_dsp_CREG_pm &pm) auto &st = pm.st_microchip_dsp_packC; log_debug("Analysing %s.%s for Microchip DSP packing (REG_C).\n", pm.module, st.dsp); - log_debug("ffC: %s\n", log_id(st.ffC, "--")); + log_debug("ffC: %s\n", st.ffC ? st.ffC->name.unescape() : "--"); Cell *cell = st.dsp; diff --git a/techlibs/xilinx/xilinx_dsp.cc b/techlibs/xilinx/xilinx_dsp.cc index 94fad6453..5c81bff22 100644 --- a/techlibs/xilinx/xilinx_dsp.cc +++ b/techlibs/xilinx/xilinx_dsp.cc @@ -262,20 +262,20 @@ void xilinx_dsp_pack(xilinx_dsp_pm &pm) log("Analysing %s.%s for Xilinx DSP packing.\n", pm.module, st.dsp); - log_debug("preAdd: %s\n", log_id(st.preAdd, "--")); - log_debug("preSub: %s\n", log_id(st.preSub, "--")); - log_debug("ffAD: %s\n", log_id(st.ffAD, "--")); - log_debug("ffA2: %s\n", log_id(st.ffA2, "--")); - log_debug("ffA1: %s\n", log_id(st.ffA1, "--")); - log_debug("ffB2: %s\n", log_id(st.ffB2, "--")); - log_debug("ffB1: %s\n", log_id(st.ffB1, "--")); - log_debug("ffD: %s\n", log_id(st.ffD, "--")); - log_debug("dsp: %s\n", log_id(st.dsp, "--")); - log_debug("ffM: %s\n", log_id(st.ffM, "--")); - log_debug("postAdd: %s\n", log_id(st.postAdd, "--")); - log_debug("postAddMux: %s\n", log_id(st.postAddMux, "--")); - log_debug("ffP: %s\n", log_id(st.ffP, "--")); - log_debug("overflow: %s\n", log_id(st.overflow, "--")); + log_debug("preAdd: %s\n", st.preAdd ? st.preAdd->name.unescape() : "--"); + log_debug("preSub: %s\n", st.preSub ? st.preSub->name.unescape() : "--"); + log_debug("ffAD: %s\n", st.ffAD ? st.ffAD->name.unescape() : "--"); + log_debug("ffA2: %s\n", st.ffA2 ? st.ffA2->name.unescape() : "--"); + log_debug("ffA1: %s\n", st.ffA1 ? st.ffA1->name.unescape() : "--"); + log_debug("ffB2: %s\n", st.ffB2 ? st.ffB2->name.unescape() : "--"); + log_debug("ffB1: %s\n", st.ffB1 ? st.ffB1->name.unescape() : "--"); + log_debug("ffD: %s\n", st.ffD ? st.ffD->name.unescape() : "--"); + log_debug("dsp: %s\n", st.dsp ? st.dsp->name.unescape() : "--"); + log_debug("ffM: %s\n", st.ffM ? st.ffM->name.unescape() : "--"); + log_debug("postAdd: %s\n", st.postAdd ? st.postAdd->name.unescape() : "--"); + log_debug("postAddMux: %s\n", st.postAddMux ? st.postAddMux->name.unescape() : "--"); + log_debug("ffP: %s\n", st.ffP ? st.ffP->name.unescape() : "--"); + log_debug("overflow: %s\n", st.overflow ? st.overflow->name.unescape() : "--"); Cell *cell = st.dsp; @@ -495,17 +495,17 @@ void xilinx_dsp48a_pack(xilinx_dsp48a_pm &pm) log("Analysing %s.%s for Xilinx DSP48A/DSP48A1 packing.\n", pm.module, st.dsp); - log_debug("preAdd: %s\n", log_id(st.preAdd, "--")); - log_debug("ffA1: %s\n", log_id(st.ffA1, "--")); - log_debug("ffA0: %s\n", log_id(st.ffA0, "--")); - log_debug("ffB1: %s\n", log_id(st.ffB1, "--")); - log_debug("ffB0: %s\n", log_id(st.ffB0, "--")); - log_debug("ffD: %s\n", log_id(st.ffD, "--")); - log_debug("dsp: %s\n", log_id(st.dsp, "--")); - log_debug("ffM: %s\n", log_id(st.ffM, "--")); - log_debug("postAdd: %s\n", log_id(st.postAdd, "--")); - log_debug("postAddMux: %s\n", log_id(st.postAddMux, "--")); - log_debug("ffP: %s\n", log_id(st.ffP, "--")); + log_debug("preAdd: %s\n", st.preAdd ? st.preAdd->name.unescape() : "--"); + log_debug("ffA1: %s\n", st.ffA1 ? st.ffA1->name.unescape() : "--"); + log_debug("ffA0: %s\n", st.ffA0 ? st.ffA0->name.unescape() : "--"); + log_debug("ffB1: %s\n", st.ffB1 ? st.ffB1->name.unescape() : "--"); + log_debug("ffB0: %s\n", st.ffB0 ? st.ffB0->name.unescape() : "--"); + log_debug("ffD: %s\n", st.ffD ? st.ffD->name.unescape() : "--"); + log_debug("dsp: %s\n", st.dsp ? st.dsp->name.unescape() : "--"); + log_debug("ffM: %s\n", st.ffM ? st.ffM->name.unescape() : "--"); + log_debug("postAdd: %s\n", st.postAdd ? st.postAdd->name.unescape() : "--"); + log_debug("postAddMux: %s\n", st.postAddMux ? st.postAddMux->name.unescape() : "--"); + log_debug("ffP: %s\n", st.ffP ? st.ffP->name.unescape() : "--"); Cell *cell = st.dsp; SigSpec &opmode = cell->connections_.at(ID(OPMODE)); @@ -672,7 +672,7 @@ void xilinx_dsp_packC(xilinx_dsp_CREG_pm &pm) auto &st = pm.st_xilinx_dsp_packC; log_debug("Analysing %s.%s for Xilinx DSP packing (CREG).\n", pm.module, st.dsp); - log_debug("ffC: %s\n", log_id(st.ffC, "--")); + log_debug("ffC: %s\n", st.ffC ? st.ffC->name.unescape() : "--"); Cell *cell = st.dsp;