diff options
Diffstat (limited to 'passes/techmap')
29 files changed, 1673 insertions, 1271 deletions
| diff --git a/passes/techmap/Makefile.inc b/passes/techmap/Makefile.inc index 56f05eca4..631a80aa5 100644 --- a/passes/techmap/Makefile.inc +++ b/passes/techmap/Makefile.inc @@ -16,6 +16,7 @@ endif  ifneq ($(SMALL),1)  OBJS += passes/techmap/iopadmap.o +OBJS += passes/techmap/clkbufmap.o  OBJS += passes/techmap/hilomap.o  OBJS += passes/techmap/extract.o  OBJS += passes/techmap/extract_fa.o diff --git a/passes/techmap/abc.cc b/passes/techmap/abc.cc index 9f963a8bf..b29480e26 100644 --- a/passes/techmap/abc.cc +++ b/passes/techmap/abc.cc @@ -166,39 +166,39 @@ void mark_port(RTLIL::SigSpec sig)  void extract_cell(RTLIL::Cell *cell, bool keepff)  { -	if (cell->type.in("$_DFF_N_", "$_DFF_P_")) +	if (cell->type.in(ID($_DFF_N_), ID($_DFF_P_)))  	{ -		if (clk_polarity != (cell->type == "$_DFF_P_")) +		if (clk_polarity != (cell->type == ID($_DFF_P_)))  			return; -		if (clk_sig != assign_map(cell->getPort("\\C"))) +		if (clk_sig != assign_map(cell->getPort(ID(C))))  			return;  		if (GetSize(en_sig) != 0)  			return;  		goto matching_dff;  	} -	if (cell->type.in("$_DFFE_NN_", "$_DFFE_NP_", "$_DFFE_PN_", "$_DFFE_PP_")) +	if (cell->type.in(ID($_DFFE_NN_), ID($_DFFE_NP_), ID($_DFFE_PN_), ID($_DFFE_PP_)))  	{ -		if (clk_polarity != cell->type.in("$_DFFE_PN_", "$_DFFE_PP_")) +		if (clk_polarity != cell->type.in(ID($_DFFE_PN_), ID($_DFFE_PP_)))  			return; -		if (en_polarity != cell->type.in("$_DFFE_NP_", "$_DFFE_PP_")) +		if (en_polarity != cell->type.in(ID($_DFFE_NP_), ID($_DFFE_PP_)))  			return; -		if (clk_sig != assign_map(cell->getPort("\\C"))) +		if (clk_sig != assign_map(cell->getPort(ID(C))))  			return; -		if (en_sig != assign_map(cell->getPort("\\E"))) +		if (en_sig != assign_map(cell->getPort(ID(E))))  			return;  		goto matching_dff;  	}  	if (0) {  	matching_dff: -		RTLIL::SigSpec sig_d = cell->getPort("\\D"); -		RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +		RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +		RTLIL::SigSpec sig_q = cell->getPort(ID(Q));  		if (keepff)  			for (auto &c : sig_q.chunks())  				if (c.wire != NULL) -					c.wire->attributes["\\keep"] = 1; +					c.wire->attributes[ID::keep] = 1;  		assign_map.apply(sig_d);  		assign_map.apply(sig_q); @@ -209,25 +209,25 @@ void extract_cell(RTLIL::Cell *cell, bool keepff)  		return;  	} -	if (cell->type.in("$_BUF_", "$_NOT_")) +	if (cell->type.in(ID($_BUF_), ID($_NOT_)))  	{ -		RTLIL::SigSpec sig_a = cell->getPort("\\A"); -		RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +		RTLIL::SigSpec sig_a = cell->getPort(ID::A); +		RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  		assign_map.apply(sig_a);  		assign_map.apply(sig_y); -		map_signal(sig_y, cell->type == "$_BUF_" ? G(BUF) : G(NOT), map_signal(sig_a)); +		map_signal(sig_y, cell->type == ID($_BUF_) ? G(BUF) : G(NOT), map_signal(sig_a));  		module->remove(cell);  		return;  	} -	if (cell->type.in("$_AND_", "$_NAND_", "$_OR_", "$_NOR_", "$_XOR_", "$_XNOR_", "$_ANDNOT_", "$_ORNOT_")) +	if (cell->type.in(ID($_AND_), ID($_NAND_), ID($_OR_), ID($_NOR_), ID($_XOR_), ID($_XNOR_), ID($_ANDNOT_), ID($_ORNOT_)))  	{ -		RTLIL::SigSpec sig_a = cell->getPort("\\A"); -		RTLIL::SigSpec sig_b = cell->getPort("\\B"); -		RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +		RTLIL::SigSpec sig_a = cell->getPort(ID::A); +		RTLIL::SigSpec sig_b = cell->getPort(ID::B); +		RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  		assign_map.apply(sig_a);  		assign_map.apply(sig_b); @@ -236,21 +236,21 @@ void extract_cell(RTLIL::Cell *cell, bool keepff)  		int mapped_a = map_signal(sig_a);  		int mapped_b = map_signal(sig_b); -		if (cell->type == "$_AND_") +		if (cell->type == ID($_AND_))  			map_signal(sig_y, G(AND), mapped_a, mapped_b); -		else if (cell->type == "$_NAND_") +		else if (cell->type == ID($_NAND_))  			map_signal(sig_y, G(NAND), mapped_a, mapped_b); -		else if (cell->type == "$_OR_") +		else if (cell->type == ID($_OR_))  			map_signal(sig_y, G(OR), mapped_a, mapped_b); -		else if (cell->type == "$_NOR_") +		else if (cell->type == ID($_NOR_))  			map_signal(sig_y, G(NOR), mapped_a, mapped_b); -		else if (cell->type == "$_XOR_") +		else if (cell->type == ID($_XOR_))  			map_signal(sig_y, G(XOR), mapped_a, mapped_b); -		else if (cell->type == "$_XNOR_") +		else if (cell->type == ID($_XNOR_))  			map_signal(sig_y, G(XNOR), mapped_a, mapped_b); -		else if (cell->type == "$_ANDNOT_") +		else if (cell->type == ID($_ANDNOT_))  			map_signal(sig_y, G(ANDNOT), mapped_a, mapped_b); -		else if (cell->type == "$_ORNOT_") +		else if (cell->type == ID($_ORNOT_))  			map_signal(sig_y, G(ORNOT), mapped_a, mapped_b);  		else  			log_abort(); @@ -259,12 +259,12 @@ void extract_cell(RTLIL::Cell *cell, bool keepff)  		return;  	} -	if (cell->type.in("$_MUX_", "$_NMUX_")) +	if (cell->type.in(ID($_MUX_), ID($_NMUX_)))  	{ -		RTLIL::SigSpec sig_a = cell->getPort("\\A"); -		RTLIL::SigSpec sig_b = cell->getPort("\\B"); -		RTLIL::SigSpec sig_s = cell->getPort("\\S"); -		RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +		RTLIL::SigSpec sig_a = cell->getPort(ID::A); +		RTLIL::SigSpec sig_b = cell->getPort(ID::B); +		RTLIL::SigSpec sig_s = cell->getPort(ID(S)); +		RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  		assign_map.apply(sig_a);  		assign_map.apply(sig_b); @@ -275,18 +275,18 @@ void extract_cell(RTLIL::Cell *cell, bool keepff)  		int mapped_b = map_signal(sig_b);  		int mapped_s = map_signal(sig_s); -		map_signal(sig_y, cell->type == "$_MUX_" ? G(MUX) : G(NMUX), mapped_a, mapped_b, mapped_s); +		map_signal(sig_y, cell->type == ID($_MUX_) ? G(MUX) : G(NMUX), mapped_a, mapped_b, mapped_s);  		module->remove(cell);  		return;  	} -	if (cell->type.in("$_AOI3_", "$_OAI3_")) +	if (cell->type.in(ID($_AOI3_), ID($_OAI3_)))  	{ -		RTLIL::SigSpec sig_a = cell->getPort("\\A"); -		RTLIL::SigSpec sig_b = cell->getPort("\\B"); -		RTLIL::SigSpec sig_c = cell->getPort("\\C"); -		RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +		RTLIL::SigSpec sig_a = cell->getPort(ID::A); +		RTLIL::SigSpec sig_b = cell->getPort(ID::B); +		RTLIL::SigSpec sig_c = cell->getPort(ID(C)); +		RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  		assign_map.apply(sig_a);  		assign_map.apply(sig_b); @@ -297,19 +297,19 @@ void extract_cell(RTLIL::Cell *cell, bool keepff)  		int mapped_b = map_signal(sig_b);  		int mapped_c = map_signal(sig_c); -		map_signal(sig_y, cell->type == "$_AOI3_" ? G(AOI3) : G(OAI3), mapped_a, mapped_b, mapped_c); +		map_signal(sig_y, cell->type == ID($_AOI3_) ? G(AOI3) : G(OAI3), mapped_a, mapped_b, mapped_c);  		module->remove(cell);  		return;  	} -	if (cell->type.in("$_AOI4_", "$_OAI4_")) +	if (cell->type.in(ID($_AOI4_), ID($_OAI4_)))  	{ -		RTLIL::SigSpec sig_a = cell->getPort("\\A"); -		RTLIL::SigSpec sig_b = cell->getPort("\\B"); -		RTLIL::SigSpec sig_c = cell->getPort("\\C"); -		RTLIL::SigSpec sig_d = cell->getPort("\\D"); -		RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +		RTLIL::SigSpec sig_a = cell->getPort(ID::A); +		RTLIL::SigSpec sig_b = cell->getPort(ID::B); +		RTLIL::SigSpec sig_c = cell->getPort(ID(C)); +		RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +		RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  		assign_map.apply(sig_a);  		assign_map.apply(sig_b); @@ -322,7 +322,7 @@ void extract_cell(RTLIL::Cell *cell, bool keepff)  		int mapped_c = map_signal(sig_c);  		int mapped_d = map_signal(sig_d); -		map_signal(sig_y, cell->type == "$_AOI4_" ? G(AOI4) : G(OAI4), mapped_a, mapped_b, mapped_c, mapped_d); +		map_signal(sig_y, cell->type == ID($_AOI4_) ? G(AOI4) : G(OAI4), mapped_a, mapped_b, mapped_c, mapped_d);  		module->remove(cell);  		return; @@ -352,23 +352,20 @@ std::string remap_name(RTLIL::IdString abc_name, RTLIL::Wire **orig_wire = nullp  				auto sig = signal_list.at(sid);  				if (sig.bit.wire != nullptr)  				{ -					std::stringstream sstr; -					sstr << "$abc$" << map_autoidx << "$" << sig.bit.wire->name.substr(1); +					std::string s = stringf("$abc$%d$%s", map_autoidx, sig.bit.wire->name.c_str()+1);  					if (sig.bit.wire->width != 1) -						sstr << "[" << sig.bit.offset << "]"; +						s += stringf("[%d]", sig.bit.offset);  					if (isnew) -						sstr << "_new"; -					sstr << postfix; +						s += "_new"; +					s += postfix;  					if (orig_wire != nullptr)  						*orig_wire = sig.bit.wire; -					return sstr.str(); +					return s;  				}  			}  		}  	} -	std::stringstream sstr; -	sstr << "$abc$" << map_autoidx << "$" << abc_name.substr(1); -	return sstr.str(); +	return stringf("$abc$%d$%s", map_autoidx, abc_name.c_str()+1);  }  void dump_loop_graph(FILE *f, int &nr, std::map<int, std::set<int>> &edges, std::set<int> &workpool, std::vector<int> &in_counts) @@ -790,7 +787,7 @@ void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std::strin  		extract_cell(c, keepff);  	for (auto &wire_it : module->wires_) { -		if (wire_it.second->port_id > 0 || wire_it.second->get_bool_attribute("\\keep")) +		if (wire_it.second->port_id > 0 || wire_it.second->get_bool_attribute(ID::keep))  			mark_port(RTLIL::SigSpec(wire_it.second));  	} @@ -939,42 +936,42 @@ void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std::strin  			log_error("Opening %s for writing failed: %s\n", buffer.c_str(), strerror(errno));  		fprintf(f, "GATE ZERO    1 Y=CONST0;\n");  		fprintf(f, "GATE ONE     1 Y=CONST1;\n"); -		fprintf(f, "GATE BUF    %d Y=A;                  PIN * NONINV  1 999 1 0 1 0\n", cell_cost.at("$_BUF_")); -		fprintf(f, "GATE NOT    %d Y=!A;                 PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_NOT_")); +		fprintf(f, "GATE BUF    %d Y=A;                  PIN * NONINV  1 999 1 0 1 0\n", cell_cost.at(ID($_BUF_))); +		fprintf(f, "GATE NOT    %d Y=!A;                 PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_NOT_)));  		if (enabled_gates.count("AND")) -			fprintf(f, "GATE AND    %d Y=A*B;                PIN * NONINV  1 999 1 0 1 0\n", cell_cost.at("$_AND_")); +			fprintf(f, "GATE AND    %d Y=A*B;                PIN * NONINV  1 999 1 0 1 0\n", cell_cost.at(ID($_AND_)));  		if (enabled_gates.count("NAND")) -			fprintf(f, "GATE NAND   %d Y=!(A*B);             PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_NAND_")); +			fprintf(f, "GATE NAND   %d Y=!(A*B);             PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_NAND_)));  		if (enabled_gates.count("OR")) -			fprintf(f, "GATE OR     %d Y=A+B;                PIN * NONINV  1 999 1 0 1 0\n", cell_cost.at("$_OR_")); +			fprintf(f, "GATE OR     %d Y=A+B;                PIN * NONINV  1 999 1 0 1 0\n", cell_cost.at(ID($_OR_)));  		if (enabled_gates.count("NOR")) -			fprintf(f, "GATE NOR    %d Y=!(A+B);             PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_NOR_")); +			fprintf(f, "GATE NOR    %d Y=!(A+B);             PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_NOR_)));  		if (enabled_gates.count("XOR")) -			fprintf(f, "GATE XOR    %d Y=(A*!B)+(!A*B);      PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at("$_XOR_")); +			fprintf(f, "GATE XOR    %d Y=(A*!B)+(!A*B);      PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at(ID($_XOR_)));  		if (enabled_gates.count("XNOR")) -			fprintf(f, "GATE XNOR   %d Y=(A*B)+(!A*!B);      PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at("$_XNOR_")); +			fprintf(f, "GATE XNOR   %d Y=(A*B)+(!A*!B);      PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at(ID($_XNOR_)));  		if (enabled_gates.count("ANDNOT")) -			fprintf(f, "GATE ANDNOT %d Y=A*!B;               PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at("$_ANDNOT_")); +			fprintf(f, "GATE ANDNOT %d Y=A*!B;               PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at(ID($_ANDNOT_)));  		if (enabled_gates.count("ORNOT")) -			fprintf(f, "GATE ORNOT  %d Y=A+!B;               PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at("$_ORNOT_")); +			fprintf(f, "GATE ORNOT  %d Y=A+!B;               PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at(ID($_ORNOT_)));  		if (enabled_gates.count("AOI3")) -			fprintf(f, "GATE AOI3   %d Y=!((A*B)+C);         PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_AOI3_")); +			fprintf(f, "GATE AOI3   %d Y=!((A*B)+C);         PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_AOI3_)));  		if (enabled_gates.count("OAI3")) -			fprintf(f, "GATE OAI3   %d Y=!((A+B)*C);         PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_OAI3_")); +			fprintf(f, "GATE OAI3   %d Y=!((A+B)*C);         PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_OAI3_)));  		if (enabled_gates.count("AOI4")) -			fprintf(f, "GATE AOI4   %d Y=!((A*B)+(C*D));     PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_AOI4_")); +			fprintf(f, "GATE AOI4   %d Y=!((A*B)+(C*D));     PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_AOI4_)));  		if (enabled_gates.count("OAI4")) -			fprintf(f, "GATE OAI4   %d Y=!((A+B)*(C+D));     PIN * INV     1 999 1 0 1 0\n", cell_cost.at("$_OAI4_")); +			fprintf(f, "GATE OAI4   %d Y=!((A+B)*(C+D));     PIN * INV     1 999 1 0 1 0\n", cell_cost.at(ID($_OAI4_)));  		if (enabled_gates.count("MUX")) -			fprintf(f, "GATE MUX    %d Y=(A*B)+(S*B)+(!S*A); PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at("$_MUX_")); +			fprintf(f, "GATE MUX    %d Y=(A*B)+(S*B)+(!S*A); PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at(ID($_MUX_)));  		if (enabled_gates.count("NMUX")) -			fprintf(f, "GATE NMUX   %d Y=!((A*B)+(S*B)+(!S*A)); PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at("$_NMUX_")); +			fprintf(f, "GATE NMUX   %d Y=!((A*B)+(S*B)+(!S*A)); PIN * UNKNOWN 1 999 1 0 1 0\n", cell_cost.at(ID($_NMUX_)));  		if (map_mux4) -			fprintf(f, "GATE MUX4   %d Y=(!S*!T*A)+(S*!T*B)+(!S*T*C)+(S*T*D); PIN * UNKNOWN 1 999 1 0 1 0\n", 2*cell_cost.at("$_MUX_")); +			fprintf(f, "GATE MUX4   %d Y=(!S*!T*A)+(S*!T*B)+(!S*T*C)+(S*T*D); PIN * UNKNOWN 1 999 1 0 1 0\n", 2*cell_cost.at(ID($_MUX_)));  		if (map_mux8) -			fprintf(f, "GATE MUX8   %d Y=(!S*!T*!U*A)+(S*!T*!U*B)+(!S*T*!U*C)+(S*T*!U*D)+(!S*!T*U*E)+(S*!T*U*F)+(!S*T*U*G)+(S*T*U*H); PIN * UNKNOWN 1 999 1 0 1 0\n", 4*cell_cost.at("$_MUX_")); +			fprintf(f, "GATE MUX8   %d Y=(!S*!T*!U*A)+(S*!T*!U*B)+(!S*T*!U*C)+(S*T*!U*D)+(!S*!T*U*E)+(S*!T*U*F)+(!S*T*U*G)+(S*T*U*H); PIN * UNKNOWN 1 999 1 0 1 0\n", 4*cell_cost.at(ID($_MUX_)));  		if (map_mux16) -			fprintf(f, "GATE MUX16  %d Y=(!S*!T*!U*!V*A)+(S*!T*!U*!V*B)+(!S*T*!U*!V*C)+(S*T*!U*!V*D)+(!S*!T*U*!V*E)+(S*!T*U*!V*F)+(!S*T*U*!V*G)+(S*T*U*!V*H)+(!S*!T*!U*V*I)+(S*!T*!U*V*J)+(!S*T*!U*V*K)+(S*T*!U*V*L)+(!S*!T*U*V*M)+(S*!T*U*V*N)+(!S*T*U*V*O)+(S*T*U*V*P); PIN * UNKNOWN 1 999 1 0 1 0\n", 8*cell_cost.at("$_MUX_")); +			fprintf(f, "GATE MUX16  %d Y=(!S*!T*!U*!V*A)+(S*!T*!U*!V*B)+(!S*T*!U*!V*C)+(S*T*!U*!V*D)+(!S*!T*U*!V*E)+(S*!T*U*!V*F)+(!S*T*U*!V*G)+(S*T*U*!V*H)+(!S*!T*!U*V*I)+(S*!T*!U*V*J)+(!S*T*!U*V*K)+(S*T*!U*V*L)+(!S*!T*U*V*M)+(S*!T*U*V*N)+(!S*T*U*V*O)+(S*T*U*V*P); PIN * UNKNOWN 1 999 1 0 1 0\n", 8*cell_cost.at(ID($_MUX_)));  		fclose(f);  		if (!lut_costs.empty()) { @@ -1019,21 +1016,21 @@ void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std::strin  		bool builtin_lib = liberty_file.empty();  		RTLIL::Design *mapped_design = new RTLIL::Design; -		parse_blif(mapped_design, ifs, builtin_lib ? "\\DFF" : "\\_dff_", false, sop_mode); +		parse_blif(mapped_design, ifs, builtin_lib ? ID(DFF) : ID(_dff_), false, sop_mode);  		ifs.close();  		log_header(design, "Re-integrating ABC results.\n"); -		RTLIL::Module *mapped_mod = mapped_design->modules_["\\netlist"]; +		RTLIL::Module *mapped_mod = mapped_design->modules_[ID(netlist)];  		if (mapped_mod == NULL)  			log_error("ABC output file does not contain a module `netlist'.\n");  		for (auto &it : mapped_mod->wires_) {  			RTLIL::Wire *w = it.second;  			RTLIL::Wire *orig_wire = nullptr;  			RTLIL::Wire *wire = module->addWire(remap_name(w->name, &orig_wire)); -			if (orig_wire != nullptr && orig_wire->attributes.count("\\src")) -				wire->attributes["\\src"] = orig_wire->attributes["\\src"]; -			if (markgroups) wire->attributes["\\abcgroup"] = map_autoidx; +			if (orig_wire != nullptr && orig_wire->attributes.count(ID(src))) +				wire->attributes[ID(src)] = orig_wire->attributes[ID(src)]; +			if (markgroups) wire->attributes[ID(abcgroup)] = map_autoidx;  			design->select(module, wire);  		} @@ -1043,141 +1040,140 @@ void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std::strin  			if (builtin_lib)  			{  				cell_stats[RTLIL::unescape_id(c->type)]++; -				if (c->type == "\\ZERO" || c->type == "\\ONE") { +				if (c->type.in(ID(ZERO), ID(ONE))) {  					RTLIL::SigSig conn; -					conn.first = RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)]); -					conn.second = RTLIL::SigSpec(c->type == "\\ZERO" ? 0 : 1, 1); +					conn.first = RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]); +					conn.second = RTLIL::SigSpec(c->type == ID(ZERO) ? 0 : 1, 1);  					module->connect(conn);  					continue;  				} -				if (c->type == "\\BUF") { +				if (c->type == ID(BUF)) {  					RTLIL::SigSig conn; -					conn.first = RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)]); -					conn.second = RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)]); +					conn.first = RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]); +					conn.second = RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)]);  					module->connect(conn);  					continue;  				} -				if (c->type == "\\NOT") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_NOT_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type == ID(NOT)) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), ID($_NOT_)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\AND" || c->type == "\\OR" || c->type == "\\XOR" || c->type == "\\NAND" || c->type == "\\NOR" || -						c->type == "\\XNOR" || c->type == "\\ANDNOT" || c->type == "\\ORNOT") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_" + c->type.substr(1) + "_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type.in(ID(AND), ID(OR), ID(XOR), ID(NAND), ID(NOR), ID(XNOR), ID(ANDNOT), ID(ORNOT))) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), stringf("$_%s_", c->type.c_str()+1)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\MUX" || c->type == "\\NMUX") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_" + c->type.substr(1) + "_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\S", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\S").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type.in(ID(MUX), ID(NMUX))) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), stringf("$_%s_", c->type.c_str()+1)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID(S), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(S)).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\MUX4") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_MUX4_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\C", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\C").as_wire()->name)])); -					cell->setPort("\\D", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\D").as_wire()->name)])); -					cell->setPort("\\S", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\S").as_wire()->name)])); -					cell->setPort("\\T", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\T").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type == ID(MUX4)) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), ID($_MUX4_)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID(C), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(C)).as_wire()->name)])); +					cell->setPort(ID(D), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(D)).as_wire()->name)])); +					cell->setPort(ID(S), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(S)).as_wire()->name)])); +					cell->setPort(ID(T), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(T)).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\MUX8") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_MUX8_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\C", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\C").as_wire()->name)])); -					cell->setPort("\\D", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\D").as_wire()->name)])); -					cell->setPort("\\E", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\E").as_wire()->name)])); -					cell->setPort("\\F", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\F").as_wire()->name)])); -					cell->setPort("\\G", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\G").as_wire()->name)])); -					cell->setPort("\\H", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\H").as_wire()->name)])); -					cell->setPort("\\S", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\S").as_wire()->name)])); -					cell->setPort("\\T", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\T").as_wire()->name)])); -					cell->setPort("\\U", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\U").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type == ID(MUX8)) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), ID($_MUX8_)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID(C), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(C)).as_wire()->name)])); +					cell->setPort(ID(D), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(D)).as_wire()->name)])); +					cell->setPort(ID(E), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(E)).as_wire()->name)])); +					cell->setPort(ID(F), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(F)).as_wire()->name)])); +					cell->setPort(ID(G), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(G)).as_wire()->name)])); +					cell->setPort(ID(H), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(H)).as_wire()->name)])); +					cell->setPort(ID(S), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(S)).as_wire()->name)])); +					cell->setPort(ID(T), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(T)).as_wire()->name)])); +					cell->setPort(ID(U), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(U)).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\MUX16") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_MUX16_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\C", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\C").as_wire()->name)])); -					cell->setPort("\\D", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\D").as_wire()->name)])); -					cell->setPort("\\E", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\E").as_wire()->name)])); -					cell->setPort("\\F", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\F").as_wire()->name)])); -					cell->setPort("\\G", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\G").as_wire()->name)])); -					cell->setPort("\\H", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\H").as_wire()->name)])); -					cell->setPort("\\I", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\I").as_wire()->name)])); -					cell->setPort("\\J", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\J").as_wire()->name)])); -					cell->setPort("\\K", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\K").as_wire()->name)])); -					cell->setPort("\\L", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\L").as_wire()->name)])); -					cell->setPort("\\M", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\M").as_wire()->name)])); -					cell->setPort("\\N", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\N").as_wire()->name)])); -					cell->setPort("\\O", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\O").as_wire()->name)])); -					cell->setPort("\\P", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\P").as_wire()->name)])); -					cell->setPort("\\S", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\S").as_wire()->name)])); -					cell->setPort("\\T", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\T").as_wire()->name)])); -					cell->setPort("\\U", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\U").as_wire()->name)])); -					cell->setPort("\\V", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\V").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type == ID(MUX16)) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), ID($_MUX16_)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID(C), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(C)).as_wire()->name)])); +					cell->setPort(ID(D), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(D)).as_wire()->name)])); +					cell->setPort(ID(E), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(E)).as_wire()->name)])); +					cell->setPort(ID(F), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(F)).as_wire()->name)])); +					cell->setPort(ID(G), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(G)).as_wire()->name)])); +					cell->setPort(ID(H), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(H)).as_wire()->name)])); +					cell->setPort(ID(I), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(I)).as_wire()->name)])); +					cell->setPort(ID(J), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(J)).as_wire()->name)])); +					cell->setPort(ID(K), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(K)).as_wire()->name)])); +					cell->setPort(ID(L), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(L)).as_wire()->name)])); +					cell->setPort(ID(M), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(M)).as_wire()->name)])); +					cell->setPort(ID(N), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(N)).as_wire()->name)])); +					cell->setPort(ID(O), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(O)).as_wire()->name)])); +					cell->setPort(ID(P), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(P)).as_wire()->name)])); +					cell->setPort(ID(S), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(S)).as_wire()->name)])); +					cell->setPort(ID(T), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(T)).as_wire()->name)])); +					cell->setPort(ID(U), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(U)).as_wire()->name)])); +					cell->setPort(ID(V), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(V)).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\AOI3" || c->type == "\\OAI3") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_" + c->type.substr(1) + "_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\C", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\C").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type.in(ID(AOI3), ID(OAI3))) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), stringf("$_%s_", c->type.c_str()+1)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID(C), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(C)).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\AOI4" || c->type == "\\OAI4") { -					RTLIL::Cell *cell = module->addCell(remap_name(c->name), "$_" + c->type.substr(1) + "_"); -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\A", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\A").as_wire()->name)])); -					cell->setPort("\\B", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\B").as_wire()->name)])); -					cell->setPort("\\C", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\C").as_wire()->name)])); -					cell->setPort("\\D", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\D").as_wire()->name)])); -					cell->setPort("\\Y", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)])); +				if (c->type.in(ID(AOI4), ID(OAI4))) { +					RTLIL::Cell *cell = module->addCell(remap_name(c->name), stringf("$_%s_", c->type.c_str()+1)); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID::A, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)])); +					cell->setPort(ID::B, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::B).as_wire()->name)])); +					cell->setPort(ID(C), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(C)).as_wire()->name)])); +					cell->setPort(ID(D), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(D)).as_wire()->name)])); +					cell->setPort(ID::Y, RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)]));  					design->select(module, cell);  					continue;  				} -				if (c->type == "\\DFF") { +				if (c->type == ID(DFF)) {  					log_assert(clk_sig.size() == 1);  					RTLIL::Cell *cell;  					if (en_sig.size() == 0) { -						cell = module->addCell(remap_name(c->name), clk_polarity ? "$_DFF_P_" : "$_DFF_N_"); +						cell = module->addCell(remap_name(c->name), clk_polarity ? ID($_DFF_P_) : ID($_DFF_N_));  					} else {  						log_assert(en_sig.size() == 1);  						cell = module->addCell(remap_name(c->name), stringf("$_DFFE_%c%c_", clk_polarity ? 'P' : 'N', en_polarity ? 'P' : 'N')); -						cell->setPort("\\E", en_sig); +						cell->setPort(ID(E), en_sig);  					} -					if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -					cell->setPort("\\D", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\D").as_wire()->name)])); -					cell->setPort("\\Q", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Q").as_wire()->name)])); -					cell->setPort("\\C", clk_sig); +					if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +					cell->setPort(ID(D), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(D)).as_wire()->name)])); +					cell->setPort(ID(Q), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(Q)).as_wire()->name)])); +					cell->setPort(ID(C), clk_sig);  					design->select(module, cell);  					continue;  				} @@ -1185,41 +1181,41 @@ void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std::strin  			else  				cell_stats[RTLIL::unescape_id(c->type)]++; -			if (c->type == "\\_const0_" || c->type == "\\_const1_") { +			if (c->type.in(ID(_const0_), ID(_const1_))) {  				RTLIL::SigSig conn;  				conn.first = RTLIL::SigSpec(module->wires_[remap_name(c->connections().begin()->second.as_wire()->name)]); -				conn.second = RTLIL::SigSpec(c->type == "\\_const0_" ? 0 : 1, 1); +				conn.second = RTLIL::SigSpec(c->type == ID(_const0_) ? 0 : 1, 1);  				module->connect(conn);  				continue;  			} -			if (c->type == "\\_dff_") { +			if (c->type == ID(_dff_)) {  				log_assert(clk_sig.size() == 1);  				RTLIL::Cell *cell;  				if (en_sig.size() == 0) { -					cell = module->addCell(remap_name(c->name), clk_polarity ? "$_DFF_P_" : "$_DFF_N_"); +					cell = module->addCell(remap_name(c->name), clk_polarity ? ID($_DFF_P_) : ID($_DFF_N_));  				} else {  					log_assert(en_sig.size() == 1);  					cell = module->addCell(remap_name(c->name), stringf("$_DFFE_%c%c_", clk_polarity ? 'P' : 'N', en_polarity ? 'P' : 'N')); -					cell->setPort("\\E", en_sig); +					cell->setPort(ID(E), en_sig);  				} -				if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; -				cell->setPort("\\D", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\D").as_wire()->name)])); -				cell->setPort("\\Q", RTLIL::SigSpec(module->wires_[remap_name(c->getPort("\\Q").as_wire()->name)])); -				cell->setPort("\\C", clk_sig); +				if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx; +				cell->setPort(ID(D), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(D)).as_wire()->name)])); +				cell->setPort(ID(Q), RTLIL::SigSpec(module->wires_[remap_name(c->getPort(ID(Q)).as_wire()->name)])); +				cell->setPort(ID(C), clk_sig);  				design->select(module, cell);  				continue;  			} -			if (c->type == "$lut" && GetSize(c->getPort("\\A")) == 1 && c->getParam("\\LUT").as_int() == 2) { -				SigSpec my_a = module->wires_[remap_name(c->getPort("\\A").as_wire()->name)]; -				SigSpec my_y = module->wires_[remap_name(c->getPort("\\Y").as_wire()->name)]; +			if (c->type == ID($lut) && GetSize(c->getPort(ID::A)) == 1 && c->getParam(ID(LUT)).as_int() == 2) { +				SigSpec my_a = module->wires_[remap_name(c->getPort(ID::A).as_wire()->name)]; +				SigSpec my_y = module->wires_[remap_name(c->getPort(ID::Y).as_wire()->name)];  				module->connect(my_y, my_a);  				continue;  			}  			RTLIL::Cell *cell = module->addCell(remap_name(c->name), c->type); -			if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; +			if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx;  			cell->parameters = c->parameters;  			for (auto &conn : c->connections()) {  				RTLIL::SigSpec newsig; @@ -1244,10 +1240,10 @@ void abc_module(RTLIL::Design *design, RTLIL::Module *current_module, std::strin  		if (recover_init)  			for (auto wire : mapped_mod->wires()) { -				if (wire->attributes.count("\\init")) { +				if (wire->attributes.count(ID(init))) {  					Wire *w = module->wires_[remap_name(wire->name)]; -					log_assert(w->attributes.count("\\init") == 0); -					w->attributes["\\init"] = wire->attributes.at("\\init"); +					log_assert(w->attributes.count(ID(init)) == 0); +					w->attributes[ID(init)] = wire->attributes.at(ID(init));  				}  			} @@ -1634,6 +1630,7 @@ struct AbcPass : public Pass {  					if (g == "ANDNOT") goto ok_gate;  					if (g == "ORNOT") goto ok_gate;  					if (g == "MUX") goto ok_gate; +					if (g == "NMUX") goto ok_gate;  					if (g == "AOI3") goto ok_gate;  					if (g == "OAI3") goto ok_gate;  					if (g == "AOI4") goto ok_gate; @@ -1802,9 +1799,9 @@ struct AbcPass : public Pass {  			signal_init.clear();  			for (Wire *wire : mod->wires()) -				if (wire->attributes.count("\\init")) { +				if (wire->attributes.count(ID(init))) {  					SigSpec initsig = assign_map(wire); -					Const initval = wire->attributes.at("\\init"); +					Const initval = wire->attributes.at(ID(init));  					for (int i = 0; i < GetSize(initsig) && i < GetSize(initval); i++)  						switch (initval[i]) {  							case State::S0: @@ -1861,16 +1858,16 @@ struct AbcPass : public Pass {  					}  				} -				if (cell->type.in("$_DFF_N_", "$_DFF_P_")) +				if (cell->type.in(ID($_DFF_N_), ID($_DFF_P_)))  				{ -					key = clkdomain_t(cell->type == "$_DFF_P_", assign_map(cell->getPort("\\C")), true, RTLIL::SigSpec()); +					key = clkdomain_t(cell->type == ID($_DFF_P_), assign_map(cell->getPort(ID(C))), true, RTLIL::SigSpec());  				}  				else -				if (cell->type.in("$_DFFE_NN_", "$_DFFE_NP_" "$_DFFE_PN_", "$_DFFE_PP_")) +				if (cell->type.in(ID($_DFFE_NN_), ID($_DFFE_NP_), ID($_DFFE_PN_), ID($_DFFE_PP_)))  				{ -					bool this_clk_pol = cell->type.in("$_DFFE_PN_", "$_DFFE_PP_"); -					bool this_en_pol = cell->type.in("$_DFFE_NP_", "$_DFFE_PP_"); -					key = clkdomain_t(this_clk_pol, assign_map(cell->getPort("\\C")), this_en_pol, assign_map(cell->getPort("\\E"))); +					bool this_clk_pol = cell->type.in(ID($_DFFE_PN_), ID($_DFFE_PP_)); +					bool this_en_pol = cell->type.in(ID($_DFFE_NP_), ID($_DFFE_PP_)); +					key = clkdomain_t(this_clk_pol, assign_map(cell->getPort(ID(C))), this_en_pol, assign_map(cell->getPort(ID(E))));  				}  				else  					continue; diff --git a/passes/techmap/abc9.cc b/passes/techmap/abc9.cc index 7418ed4a3..84cb2c04f 100644 --- a/passes/techmap/abc9.cc +++ b/passes/techmap/abc9.cc @@ -71,25 +71,22 @@ RTLIL::Module *module;  bool clk_polarity, en_polarity;  RTLIL::SigSpec clk_sig, en_sig; -std::string remap_name(RTLIL::IdString abc_name) +inline std::string remap_name(RTLIL::IdString abc_name)  { -	std::stringstream sstr; -	sstr << "$abc$" << map_autoidx << "$" << abc_name.substr(1); -	return sstr.str(); +	return stringf("$abc$%d$%s", map_autoidx, abc_name.c_str()+1);  } -void handle_loops(RTLIL::Design *design) +void handle_loops(RTLIL::Design *design, +		const dict<IdString,pool<IdString>> &scc_break_inputs)  {  	Pass::call(design, "scc -set_attr abc_scc_id {}"); -	dict<IdString, vector<IdString>> abc_scc_break; -  	// For every unique SCC found, (arbitrarily) find the first  	// cell in the component, and select (and mark) all its output  	// wires  	pool<RTLIL::Const> ids_seen;  	for (auto cell : module->cells()) { -		auto it = cell->attributes.find("\\abc_scc_id"); +		auto it = cell->attributes.find(ID(abc_scc_id));  		if (it != cell->attributes.end()) {  			auto r = ids_seen.insert(it->second);  			if (r.second) { @@ -109,7 +106,7 @@ void handle_loops(RTLIL::Design *design)  							log_assert(w->port_input);  							log_assert(b.offset < GetSize(w));  						} -						w->set_bool_attribute("\\abc_scc_break"); +						w->set_bool_attribute(ID(abc_scc_break));  						module->swap_names(b.wire, w);  						c.second = RTLIL::SigBit(w, b.offset);  					} @@ -118,44 +115,29 @@ void handle_loops(RTLIL::Design *design)  			cell->attributes.erase(it);  		} -		auto jt = abc_scc_break.find(cell->type); -		if (jt == abc_scc_break.end()) { -			std::vector<IdString> ports; -			RTLIL::Module* box_module = design->module(cell->type); -			if (box_module) { -				auto ports_csv = box_module->attributes.at("\\abc_scc_break", RTLIL::Const::from_string("")).decode_string(); -				for (const auto &port_name : split_tokens(ports_csv, ",")) { -					auto port_id = RTLIL::escape_id(port_name); -					auto kt = cell->connections_.find(port_id); -					if (kt == cell->connections_.end()) -						log_error("abc_scc_break attribute value '%s' does not exist as port on module '%s'\n", port_name.c_str(), log_id(box_module)); -					ports.push_back(port_id); -				} -			} -			jt = abc_scc_break.insert(std::make_pair(cell->type, std::move(ports))).first; -		} - -		for (auto port_name : jt->second) { -			RTLIL::SigSpec sig; -			auto &rhs = cell->connections_.at(port_name); -			for (auto b : rhs) { -				Wire *w = b.wire; -				if (!w) continue; -				w->port_output = true; -				w->set_bool_attribute("\\abc_scc_break"); -				w = module->wire(stringf("%s.abci", w->name.c_str())); -				if (!w) { -					w = module->addWire(stringf("%s.abci", b.wire->name.c_str()), GetSize(b.wire)); -					w->port_input = true; -				} -				else { -					log_assert(b.offset < GetSize(w)); -					log_assert(w->port_input); +		auto jt = scc_break_inputs.find(cell->type); +		if (jt != scc_break_inputs.end()) +			for (auto port_name : jt->second) { +				RTLIL::SigSpec sig; +				auto &rhs = cell->connections_.at(port_name); +				for (auto b : rhs) { +					Wire *w = b.wire; +					if (!w) continue; +					w->port_output = true; +					w->set_bool_attribute(ID(abc_scc_break)); +					w = module->wire(stringf("%s.abci", w->name.c_str())); +					if (!w) { +						w = module->addWire(stringf("%s.abci", b.wire->name.c_str()), GetSize(b.wire)); +						w->port_input = true; +					} +					else { +						log_assert(b.offset < GetSize(w)); +						log_assert(w->port_input); +					} +					sig.append(RTLIL::SigBit(w, b.offset));  				} -				sig.append(RTLIL::SigBit(w, b.offset)); +				rhs = sig;  			} -			rhs = sig; -		}  	}  	module->fixup_ports(); @@ -290,7 +272,9 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		bool cleanup, vector<int> lut_costs, bool dff_mode, std::string clk_str,  		bool /*keepff*/, std::string delay_target, std::string /*lutin_shared*/, bool fast_mode,  		bool show_tempdir, std::string box_file, std::string lut_file, -		std::string wire_delay, const dict<int,IdString> &box_lookup) +		std::string wire_delay, const dict<int,IdString> &box_lookup, +		const dict<IdString,pool<IdString>> &scc_break_inputs +)  {  	module = current_module;  	map_autoidx = autoidx++; @@ -429,7 +413,7 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		RTLIL::Selection& sel = design->selection_stack.back();  		sel.select(module); -		handle_loops(design); +		handle_loops(design, scc_break_inputs);  		Pass::call(design, "aigmap"); @@ -446,14 +430,14 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		if (ifs.fail())  			log_error("Can't open ABC output file `%s'.\n", buffer.c_str());  		buffer = stringf("%s/%s", tempdir_name.c_str(), "input.sym"); -		log_assert(!design->module("$__abc9__")); +		log_assert(!design->module(ID($__abc9__)));  		{ -			AigerReader reader(design, ifs, "$__abc9__", "" /* clk_name */, buffer.c_str() /* map_filename */, true /* wideports */); +			AigerReader reader(design, ifs, ID($__abc9__), "" /* clk_name */, buffer.c_str() /* map_filename */, true /* wideports */);  			reader.parse_xaiger();  		}  		ifs.close();  		Pass::call(design, stringf("write_verilog -noexpr -norename")); -		design->remove(design->module("$__abc9__")); +		design->remove(design->module(ID($__abc9__)));  #endif  		design->selection_stack.pop_back(); @@ -462,7 +446,7 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		// the expose operation -- remove them from PO/PI  		// and re-connecting them back together  		for (auto wire : module->wires()) { -			auto it = wire->attributes.find("\\abc_scc_break"); +			auto it = wire->attributes.find(ID(abc_scc_break));  			if (it != wire->attributes.end()) {  				wire->attributes.erase(it);  				log_assert(wire->port_output); @@ -518,9 +502,9 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  			log_error("Can't open ABC output file `%s'.\n", buffer.c_str());  		buffer = stringf("%s/%s", tempdir_name.c_str(), "input.sym"); -		log_assert(!design->module("$__abc9__")); +		log_assert(!design->module(ID($__abc9__))); -		AigerReader reader(design, ifs, "$__abc9__", "" /* clk_name */, buffer.c_str() /* map_filename */, true /* wideports */); +		AigerReader reader(design, ifs, ID($__abc9__), "" /* clk_name */, buffer.c_str() /* map_filename */, true /* wideports */);  		reader.parse_xaiger(box_lookup);  		ifs.close(); @@ -529,7 +513,7 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  #endif  		log_header(design, "Re-integrating ABC9 results.\n"); -		RTLIL::Module *mapped_mod = design->module("$__abc9__"); +		RTLIL::Module *mapped_mod = design->module(ID($__abc9__));  		if (mapped_mod == NULL)  			log_error("ABC output file does not contain a module `$__abc9__'.\n"); @@ -537,7 +521,7 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		for (auto &it : mapped_mod->wires_) {  			RTLIL::Wire *w = it.second;  			RTLIL::Wire *remap_wire = module->addWire(remap_name(w->name), GetSize(w)); -			if (markgroups) remap_wire->attributes["\\abcgroup"] = map_autoidx; +			if (markgroups) remap_wire->attributes[ID(abcgroup)] = map_autoidx;  			if (w->port_output) {  				RTLIL::Wire *wire = module->wire(w->name);  				log_assert(wire); @@ -559,14 +543,14 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		vector<RTLIL::Cell*> boxes;  		for (const auto &it : module->cells_) {  			auto cell = it.second; -			if (cell->type.in("$_AND_", "$_NOT_")) { +			if (cell->type.in(ID($_AND_), ID($_NOT_))) {  				module->remove(cell);  				continue;  			}  			auto jt = abc_box.find(cell->type);  			if (jt == abc_box.end()) {  				RTLIL::Module* box_module = design->module(cell->type); -				jt = abc_box.insert(std::make_pair(cell->type, box_module && box_module->attributes.count("\\abc_box_id"))).first; +				jt = abc_box.insert(std::make_pair(cell->type, box_module && box_module->attributes.count(ID(abc_box_id)))).first;  			}  			if (jt->second)  				boxes.emplace_back(cell); @@ -577,20 +561,20 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  		dict<RTLIL::Cell*,RTLIL::Cell*> not2drivers;  		dict<SigBit, std::vector<RTLIL::Cell*>> bit2sinks; -		std::map<std::string, int> cell_stats; +		std::map<IdString, int> cell_stats;  		for (auto c : mapped_mod->cells())  		{  			toposort.node(c->name);  			RTLIL::Cell *cell = nullptr; -			if (c->type == "$_NOT_") { -				RTLIL::SigBit a_bit = c->getPort("\\A"); -				RTLIL::SigBit y_bit = c->getPort("\\Y"); +			if (c->type == ID($_NOT_)) { +				RTLIL::SigBit a_bit = c->getPort(ID::A); +				RTLIL::SigBit y_bit = c->getPort(ID::Y);  				bit_users[a_bit].insert(c->name);  				bit_drivers[y_bit].insert(c->name);  				if (!a_bit.wire) { -					c->setPort("\\Y", module->addWire(NEW_ID)); +					c->setPort(ID::Y, module->addWire(NEW_ID));  					RTLIL::Wire *wire = module->wire(remap_name(y_bit.wire->name));  					log_assert(wire);  					module->connect(RTLIL::SigBit(wire, y_bit.offset), State::S1); @@ -618,8 +602,8 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  								RTLIL::SigBit(module->wires_.at(remap_name(a_bit.wire->name)), a_bit.offset),  								RTLIL::SigBit(module->wires_.at(remap_name(y_bit.wire->name)), y_bit.offset),  								RTLIL::Const::from_string("01")); -						bit2sinks[cell->getPort("\\A")].push_back(cell); -						cell_stats["$lut"]++; +						bit2sinks[cell->getPort(ID::A)].push_back(cell); +						cell_stats[ID($lut)]++;  					}  					else  						not2drivers[c] = driver_lut; @@ -627,18 +611,18 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  				}  				else  					log_abort(); -				if (cell && markgroups) cell->attributes["\\abcgroup"] = map_autoidx; +				if (cell && markgroups) cell->attributes[ID(abcgroup)] = map_autoidx;  				continue;  			} -			cell_stats[RTLIL::unescape_id(c->type)]++; +			cell_stats[c->type]++;  			RTLIL::Cell *existing_cell = nullptr; -			if (c->type == "$lut") { -				if (GetSize(c->getPort("\\A")) == 1 && c->getParam("\\LUT") == RTLIL::Const::from_string("01")) { -					SigSpec my_a = module->wires_.at(remap_name(c->getPort("\\A").as_wire()->name)); -					SigSpec my_y = module->wires_.at(remap_name(c->getPort("\\Y").as_wire()->name)); +			if (c->type == ID($lut)) { +				if (GetSize(c->getPort(ID::A)) == 1 && c->getParam(ID(LUT)) == RTLIL::Const::from_string("01")) { +					SigSpec my_a = module->wires_.at(remap_name(c->getPort(ID::A).as_wire()->name)); +					SigSpec my_y = module->wires_.at(remap_name(c->getPort(ID::Y).as_wire()->name));  					module->connect(my_y, my_a); -					if (markgroups) c->attributes["\\abcgroup"] = map_autoidx; +					if (markgroups) c->attributes[ID(abcgroup)] = map_autoidx;  					log_abort();  					continue;  				} @@ -651,7 +635,7 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  				module->swap_names(cell, existing_cell);  			} -			if (markgroups) cell->attributes["\\abcgroup"] = map_autoidx; +			if (markgroups) cell->attributes[ID(abcgroup)] = map_autoidx;  			if (existing_cell) {  				cell->parameters = existing_cell->parameters;  				cell->attributes = existing_cell->attributes; @@ -741,20 +725,20 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  				for (auto driver_cell : bit_drivers.at(it.first))  				for (auto user_cell : it.second)  					toposort.edge(driver_cell, user_cell); -		bool no_loops = toposort.sort(); +		bool no_loops YS_ATTRIBUTE(unused) = toposort.sort();  		log_assert(no_loops);  		for (auto ii = toposort.sorted.rbegin(); ii != toposort.sorted.rend(); ii++) {  			RTLIL::Cell *not_cell = mapped_mod->cell(*ii);  			log_assert(not_cell); -			if (not_cell->type != "$_NOT_") +			if (not_cell->type != ID($_NOT_))  				continue;  			auto it = not2drivers.find(not_cell);  			if (it == not2drivers.end())  				continue;  			RTLIL::Cell *driver_lut = it->second; -			RTLIL::SigBit a_bit = not_cell->getPort("\\A"); -			RTLIL::SigBit y_bit = not_cell->getPort("\\Y"); +			RTLIL::SigBit a_bit = not_cell->getPort(ID::A); +			RTLIL::SigBit y_bit = not_cell->getPort(ID::Y);  			RTLIL::Const driver_mask;  			a_bit.wire = module->wires_.at(remap_name(a_bit.wire->name)); @@ -765,13 +749,13 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  				goto clone_lut;  			for (auto sink_cell : jt->second) -				if (sink_cell->type != "$lut") +				if (sink_cell->type != ID($lut))  					goto clone_lut;  			// Push downstream LUTs past inverter  			for (auto sink_cell : jt->second) { -				SigSpec A = sink_cell->getPort("\\A"); -				RTLIL::Const mask = sink_cell->getParam("\\LUT"); +				SigSpec A = sink_cell->getPort(ID::A); +				RTLIL::Const mask = sink_cell->getParam(ID(LUT));  				int index = 0;  				for (; index < GetSize(A); index++)  					if (A[index] == a_bit) @@ -784,8 +768,8 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  					i += 1 << (index+1);  				}  				A[index] = y_bit; -				sink_cell->setPort("\\A", A); -				sink_cell->setParam("\\LUT", mask); +				sink_cell->setPort(ID::A, A); +				sink_cell->setParam(ID(LUT), mask);  			}  			// Since we have rewritten all sinks (which we know @@ -794,16 +778,16 @@ void abc9_module(RTLIL::Design *design, RTLIL::Module *current_module, std::stri  			// that the original driving LUT will become dangling  			// and get cleaned away  clone_lut: -			driver_mask = driver_lut->getParam("\\LUT"); +			driver_mask = driver_lut->getParam(ID(LUT));  			for (auto &b : driver_mask.bits) {  				if (b == RTLIL::State::S0) b = RTLIL::State::S1;  				else if (b == RTLIL::State::S1) b = RTLIL::State::S0;  			}  			auto cell = module->addLut(NEW_ID, -					driver_lut->getPort("\\A"), +					driver_lut->getPort(ID::A),  					y_bit,  					driver_mask); -			for (auto &bit : cell->connections_.at("\\A")) { +			for (auto &bit : cell->connections_.at(ID::A)) {  				bit.wire = module->wires_.at(remap_name(bit.wire->name));  				bit2sinks[bit].push_back(cell);  			} @@ -1083,8 +1067,9 @@ struct Abc9Pass : public Pass {  		extra_args(args, argidx, design);  		dict<int,IdString> box_lookup; +		dict<IdString,pool<IdString>> scc_break_inputs;  		for (auto m : design->modules()) { -			auto it = m->attributes.find("\\abc_box_id"); +			auto it = m->attributes.find(ID(abc_box_id));  			if (it == m->attributes.end())  				continue;  			if (m->name.begins_with("$paramod")) @@ -1095,11 +1080,61 @@ struct Abc9Pass : public Pass {  				log_error("Module '%s' has the same abc_box_id = %d value as '%s'.\n",  						log_id(m), id, log_id(r.first->second));  			log_assert(r.second); + +			RTLIL::Wire *carry_in = nullptr, *carry_out = nullptr; +			for (auto p : m->ports) { +				auto w = m->wire(p); +				log_assert(w); +				if (w->port_input) { +					if (w->attributes.count(ID(abc_scc_break))) +						scc_break_inputs[m->name].insert(p); +					if (w->attributes.count(ID(abc_carry))) { +						if (carry_in) +							log_error("Module '%s' contains more than one 'abc_carry' input port.\n", log_id(m)); +						carry_in = w; +					} +				} +				if (w->port_output) { +					if (w->attributes.count(ID(abc_carry))) { +						if (carry_out) +							log_error("Module '%s' contains more than one 'abc_carry' input port.\n", log_id(m)); +						carry_out = w; +					} +				} +			} +			if (carry_in || carry_out) { +				if (carry_in && !carry_out) +					log_error("Module '%s' contains an 'abc_carry' input port but no output port.\n", log_id(m)); +				if (!carry_in && carry_out) +					log_error("Module '%s' contains an 'abc_carry' output port but no input port.\n", log_id(m)); +				// Make carry_in the last PI, and carry_out the last PO +				//   since ABC requires it this way +				auto &ports = m->ports; +				for (auto it = ports.begin(); it != ports.end(); ) { +					RTLIL::Wire* w = m->wire(*it); +					log_assert(w); +					if (w == carry_in || w == carry_out) { +						it = ports.erase(it); +						continue; +					} +					if (w->port_id > carry_in->port_id) +						--w->port_id; +					if (w->port_id > carry_out->port_id) +						--w->port_id; +					log_assert(w->port_input || w->port_output); +					log_assert(ports[w->port_id-1] == w->name); +					++it; +				} +				ports.push_back(carry_in->name); +				carry_in->port_id = ports.size(); +				ports.push_back(carry_out->name); +				carry_out->port_id = ports.size(); +			}  		}  		for (auto mod : design->selected_modules())  		{ -			if (mod->attributes.count("\\abc_box_id")) +			if (mod->attributes.count(ID(abc_box_id)))  				continue;  			if (mod->processes.size() > 0) { @@ -1112,7 +1147,7 @@ struct Abc9Pass : public Pass {  			if (!dff_mode || !clk_str.empty()) {  				abc9_module(design, mod, script_file, exe_file, cleanup, lut_costs, dff_mode, clk_str, keepff,  						delay_target, lutin_shared, fast_mode, show_tempdir, -						box_file, lut_file, wire_delay, box_lookup); +						box_file, lut_file, wire_delay, box_lookup, scc_break_inputs);  				continue;  			} @@ -1153,16 +1188,16 @@ struct Abc9Pass : public Pass {  					}  				} -				if (cell->type.in("$_DFF_N_", "$_DFF_P_")) +				if (cell->type.in(ID($_DFF_N_), ID($_DFF_P_)))  				{ -					key = clkdomain_t(cell->type == "$_DFF_P_", assign_map(cell->getPort("\\C")), true, RTLIL::SigSpec()); +					key = clkdomain_t(cell->type == ID($_DFF_P_), assign_map(cell->getPort(ID(C))), true, RTLIL::SigSpec());  				}  				else -				if (cell->type.in("$_DFFE_NN_", "$_DFFE_NP_", "$_DFFE_PN_", "$_DFFE_PP_")) +				if (cell->type.in(ID($_DFFE_NN_), ID($_DFFE_NP_), ID($_DFFE_PN_), ID($_DFFE_PP_)))  				{ -					bool this_clk_pol = cell->type.in("$_DFFE_PN_", "$_DFFE_PP_"); -					bool this_en_pol = cell->type.in("$_DFFE_NP_", "$_DFFE_PP_"); -					key = clkdomain_t(this_clk_pol, assign_map(cell->getPort("\\C")), this_en_pol, assign_map(cell->getPort("\\E"))); +					bool this_clk_pol = cell->type.in(ID($_DFFE_PN_), ID($_DFFE_PP_)); +					bool this_en_pol = cell->type.in(ID($_DFFE_NP_), ID($_DFFE_PP_)); +					key = clkdomain_t(this_clk_pol, assign_map(cell->getPort(ID(C))), this_en_pol, assign_map(cell->getPort(ID(E))));  				}  				else  					continue; @@ -1258,7 +1293,7 @@ struct Abc9Pass : public Pass {  				en_sig = assign_map(std::get<3>(it.first));  				abc9_module(design, mod, script_file, exe_file, cleanup, lut_costs, !clk_sig.empty(), "$",  						keepff, delay_target, lutin_shared, fast_mode, show_tempdir, -						box_file, lut_file, wire_delay, box_lookup); +						box_file, lut_file, wire_delay, box_lookup, scc_break_inputs);  				assign_map.set(mod);  			}  		} diff --git a/passes/techmap/aigmap.cc b/passes/techmap/aigmap.cc index 2423676cb..1d5e1286b 100644 --- a/passes/techmap/aigmap.cc +++ b/passes/techmap/aigmap.cc @@ -66,10 +66,10 @@ struct AigmapPass : public Pass {  			{  				Aig aig(cell); -				if (cell->type.in("$_AND_", "$_NOT_")) +				if (cell->type.in(ID($_AND_), ID($_NOT_)))  					aig.name.clear(); -				if (nand_mode && cell->type == "$_NAND_") +				if (nand_mode && cell->type == ID($_NAND_))  					aig.name.clear();  				if (aig.name.empty()) { diff --git a/passes/techmap/alumacc.cc b/passes/techmap/alumacc.cc index 5c9e42fd4..5b168d524 100644 --- a/passes/techmap/alumacc.cc +++ b/passes/techmap/alumacc.cc @@ -61,7 +61,7 @@ struct AlumaccWorker  		RTLIL::SigSpec get_eq() {  			if (GetSize(cached_eq) == 0) -				cached_eq = alu_cell->module->ReduceAnd(NEW_ID, alu_cell->getPort("\\X"), false, alu_cell->get_src_attribute()); +				cached_eq = alu_cell->module->ReduceAnd(NEW_ID, alu_cell->getPort(ID(X)), false, alu_cell->get_src_attribute());  			return cached_eq;  		} @@ -73,7 +73,7 @@ struct AlumaccWorker  		RTLIL::SigSpec get_cf() {  			if (GetSize(cached_cf) == 0) { -				cached_cf = alu_cell->getPort("\\CO"); +				cached_cf = alu_cell->getPort(ID(CO));  				log_assert(GetSize(cached_cf) >= 1);  				cached_cf = alu_cell->module->Not(NEW_ID, cached_cf[GetSize(cached_cf)-1], false, alu_cell->get_src_attribute());  			} @@ -82,7 +82,7 @@ struct AlumaccWorker  		RTLIL::SigSpec get_of() {  			if (GetSize(cached_of) == 0) { -				cached_of = {alu_cell->getPort("\\CO"), alu_cell->getPort("\\CI")}; +				cached_of = {alu_cell->getPort(ID(CO)), alu_cell->getPort(ID(CI))};  				log_assert(GetSize(cached_of) >= 2);  				cached_of = alu_cell->module->Xor(NEW_ID, cached_of[GetSize(cached_of)-1], cached_of[GetSize(cached_of)-2]);  			} @@ -91,7 +91,7 @@ struct AlumaccWorker  		RTLIL::SigSpec get_sf() {  			if (GetSize(cached_sf) == 0) { -				cached_sf = alu_cell->getPort("\\Y"); +				cached_sf = alu_cell->getPort(ID::Y);  				cached_sf = cached_sf[GetSize(cached_sf)-1];  			}  			return cached_sf; @@ -125,7 +125,7 @@ struct AlumaccWorker  	{  		for (auto cell : module->selected_cells())  		{ -			if (!cell->type.in("$pos", "$neg", "$add", "$sub", "$mul")) +			if (!cell->type.in(ID($pos), ID($neg), ID($add), ID($sub), ID($mul)))  				continue;  			log("  creating $macc model for %s (%s).\n", log_id(cell), log_id(cell->type)); @@ -134,38 +134,38 @@ struct AlumaccWorker  			Macc::port_t new_port;  			n->cell = cell; -			n->y = sigmap(cell->getPort("\\Y")); +			n->y = sigmap(cell->getPort(ID::Y));  			n->users = 0;  			for (auto bit : n->y)  				n->users = max(n->users, bit_users.at(bit) - 1); -			if (cell->type.in("$pos", "$neg")) +			if (cell->type.in(ID($pos), ID($neg)))  			{ -				new_port.in_a = sigmap(cell->getPort("\\A")); -				new_port.is_signed = cell->getParam("\\A_SIGNED").as_bool(); -				new_port.do_subtract = cell->type == "$neg"; +				new_port.in_a = sigmap(cell->getPort(ID::A)); +				new_port.is_signed = cell->getParam(ID(A_SIGNED)).as_bool(); +				new_port.do_subtract = cell->type == ID($neg);  				n->macc.ports.push_back(new_port);  			} -			if (cell->type.in("$add", "$sub")) +			if (cell->type.in(ID($add), ID($sub)))  			{ -				new_port.in_a = sigmap(cell->getPort("\\A")); -				new_port.is_signed = cell->getParam("\\A_SIGNED").as_bool(); +				new_port.in_a = sigmap(cell->getPort(ID::A)); +				new_port.is_signed = cell->getParam(ID(A_SIGNED)).as_bool();  				new_port.do_subtract = false;  				n->macc.ports.push_back(new_port); -				new_port.in_a = sigmap(cell->getPort("\\B")); -				new_port.is_signed = cell->getParam("\\B_SIGNED").as_bool(); -				new_port.do_subtract = cell->type == "$sub"; +				new_port.in_a = sigmap(cell->getPort(ID::B)); +				new_port.is_signed = cell->getParam(ID(B_SIGNED)).as_bool(); +				new_port.do_subtract = cell->type == ID($sub);  				n->macc.ports.push_back(new_port);  			} -			if (cell->type.in("$mul")) +			if (cell->type.in(ID($mul)))  			{ -				new_port.in_a = sigmap(cell->getPort("\\A")); -				new_port.in_b = sigmap(cell->getPort("\\B")); -				new_port.is_signed = cell->getParam("\\A_SIGNED").as_bool(); +				new_port.in_a = sigmap(cell->getPort(ID::A)); +				new_port.in_b = sigmap(cell->getPort(ID::B)); +				new_port.is_signed = cell->getParam(ID(A_SIGNED)).as_bool();  				new_port.do_subtract = false;  				n->macc.ports.push_back(new_port);  			} @@ -351,7 +351,7 @@ struct AlumaccWorker  		for (auto &it : sig_macc)  		{  			auto n = it.second; -			auto cell = module->addCell(NEW_ID, "$macc"); +			auto cell = module->addCell(NEW_ID, ID($macc));  			macc_counter++; @@ -361,7 +361,7 @@ struct AlumaccWorker  			n->macc.optimize(GetSize(n->y));  			n->macc.to_cell(cell); -			cell->setPort("\\Y", n->y); +			cell->setPort(ID::Y, n->y);  			cell->fixup_parameters();  			module->remove(n->cell);  			delete n; @@ -376,9 +376,9 @@ struct AlumaccWorker  		for (auto cell : module->selected_cells())  		{ -			if (cell->type.in("$lt", "$le", "$ge", "$gt")) +			if (cell->type.in(ID($lt), ID($le), ID($ge), ID($gt)))  				lge_cells.push_back(cell); -			if (cell->type.in("$eq", "$eqx", "$ne", "$nex")) +			if (cell->type.in(ID($eq), ID($eqx), ID($ne), ID($nex)))  				eq_cells.push_back(cell);  		} @@ -386,13 +386,13 @@ struct AlumaccWorker  		{  			log("  creating $alu model for %s (%s):", log_id(cell), log_id(cell->type)); -			bool cmp_less = cell->type.in("$lt", "$le"); -			bool cmp_equal = cell->type.in("$le", "$ge"); -			bool is_signed = cell->getParam("\\A_SIGNED").as_bool(); +			bool cmp_less = cell->type.in(ID($lt), ID($le)); +			bool cmp_equal = cell->type.in(ID($le), ID($ge)); +			bool is_signed = cell->getParam(ID(A_SIGNED)).as_bool(); -			RTLIL::SigSpec A = sigmap(cell->getPort("\\A")); -			RTLIL::SigSpec B = sigmap(cell->getPort("\\B")); -			RTLIL::SigSpec Y = sigmap(cell->getPort("\\Y")); +			RTLIL::SigSpec A = sigmap(cell->getPort(ID::A)); +			RTLIL::SigSpec B = sigmap(cell->getPort(ID::B)); +			RTLIL::SigSpec Y = sigmap(cell->getPort(ID::Y));  			if (B < A && GetSize(B)) {  				cmp_less = !cmp_less; @@ -427,12 +427,12 @@ struct AlumaccWorker  		for (auto cell : eq_cells)  		{ -			bool cmp_equal = cell->type.in("$eq", "$eqx"); -			bool is_signed = cell->getParam("\\A_SIGNED").as_bool(); +			bool cmp_equal = cell->type.in(ID($eq), ID($eqx)); +			bool is_signed = cell->getParam(ID(A_SIGNED)).as_bool(); -			RTLIL::SigSpec A = sigmap(cell->getPort("\\A")); -			RTLIL::SigSpec B = sigmap(cell->getPort("\\B")); -			RTLIL::SigSpec Y = sigmap(cell->getPort("\\Y")); +			RTLIL::SigSpec A = sigmap(cell->getPort(ID::A)); +			RTLIL::SigSpec B = sigmap(cell->getPort(ID::B)); +			RTLIL::SigSpec Y = sigmap(cell->getPort(ID::Y));  			if (B < A && GetSize(B))  				std::swap(A, B); @@ -471,7 +471,7 @@ struct AlumaccWorker  				goto delete_node;  			} -			n->alu_cell = module->addCell(NEW_ID, "$alu"); +			n->alu_cell = module->addCell(NEW_ID, ID($alu));  			alu_counter++;  			log("  creating $alu cell for "); @@ -482,13 +482,13 @@ struct AlumaccWorker  			if (n->cells.size() > 0)  				n->alu_cell->set_src_attribute(n->cells[0]->get_src_attribute()); -			n->alu_cell->setPort("\\A", n->a); -			n->alu_cell->setPort("\\B", n->b); -			n->alu_cell->setPort("\\CI", GetSize(n->c) ? n->c : State::S0); -			n->alu_cell->setPort("\\BI", n->invert_b ? State::S1 : State::S0); -			n->alu_cell->setPort("\\Y", n->y); -			n->alu_cell->setPort("\\X", module->addWire(NEW_ID, GetSize(n->y))); -			n->alu_cell->setPort("\\CO", module->addWire(NEW_ID, GetSize(n->y))); +			n->alu_cell->setPort(ID::A, n->a); +			n->alu_cell->setPort(ID::B, n->b); +			n->alu_cell->setPort(ID(CI), GetSize(n->c) ? n->c : State::S0); +			n->alu_cell->setPort(ID(BI), n->invert_b ? State::S1 : State::S0); +			n->alu_cell->setPort(ID::Y, n->y); +			n->alu_cell->setPort(ID(X), module->addWire(NEW_ID, GetSize(n->y))); +			n->alu_cell->setPort(ID(CO), module->addWire(NEW_ID, GetSize(n->y)));  			n->alu_cell->fixup_parameters(n->is_signed, n->is_signed);  			for (auto &it : n->cmp) diff --git a/passes/techmap/attrmap.cc b/passes/techmap/attrmap.cc index a38638e0b..5f30817d4 100644 --- a/passes/techmap/attrmap.cc +++ b/passes/techmap/attrmap.cc @@ -143,6 +143,82 @@ void attrmap_apply(string objname, vector<std::unique_ptr<AttrmapAction>> &actio  	attributes.swap(new_attributes);  } +void log_attrmap_paramap_options() +{ +	log("    -tocase <name>\n"); +	log("        Match attribute names case-insensitively and set it to the specified\n"); +	log("        name.\n"); +	log("\n"); +	log("    -rename <old_name> <new_name>\n"); +	log("        Rename attributes as specified\n"); +	log("\n"); +	log("    -map <old_name>=<old_value> <new_name>=<new_value>\n"); +	log("        Map key/value pairs as indicated.\n"); +	log("\n"); +	log("    -imap <old_name>=<old_value> <new_name>=<new_value>\n"); +	log("        Like -map, but use case-insensitive match for <old_value> when\n"); +	log("        it is a string value.\n"); +	log("\n"); +	log("    -remove <name>=<value>\n"); +	log("        Remove attributes matching this pattern.\n"); +} + +bool parse_attrmap_paramap_options(size_t &argidx, std::vector<std::string> &args, vector<std::unique_ptr<AttrmapAction>> &actions) +{ +	std::string arg = args[argidx]; +	if (arg == "-tocase" && argidx+1 < args.size()) { +		auto action = new AttrmapTocase; +		action->name = args[++argidx]; +		actions.push_back(std::unique_ptr<AttrmapAction>(action)); +		return true; +	} +	if (arg == "-rename" && argidx+2 < args.size()) { +		auto action = new AttrmapRename; +		action->old_name = args[++argidx]; +		action->new_name = args[++argidx]; +		actions.push_back(std::unique_ptr<AttrmapAction>(action)); +		return true; +	} +	if ((arg == "-map" || arg == "-imap") && argidx+2 < args.size()) { +		string arg1 = args[++argidx]; +		string arg2 = args[++argidx]; +		string val1, val2; +		size_t p = arg1.find("="); +		if (p != string::npos) { +			val1 = arg1.substr(p+1); +			arg1 = arg1.substr(0, p); +		} +		p = arg2.find("="); +		if (p != string::npos) { +			val2 = arg2.substr(p+1); +			arg2 = arg2.substr(0, p); +		} +		auto action = new AttrmapMap; +		action->imap = (arg == "-map"); +		action->old_name = arg1; +		action->new_name = arg2; +		action->old_value = val1; +		action->new_value = val2; +		actions.push_back(std::unique_ptr<AttrmapAction>(action)); +		return true; +	} +	if (arg == "-remove" && argidx+1 < args.size()) { +		string arg1 = args[++argidx], val1; +		size_t p = arg1.find("="); +		if (p != string::npos) { +			val1 = arg1.substr(p+1); +			arg1 = arg1.substr(0, p); +		} +		auto action = new AttrmapRemove; +		action->name = arg1; +		action->has_value = (p != string::npos); +		action->value = val1; +		actions.push_back(std::unique_ptr<AttrmapAction>(action)); +		return true; +	} +	return false; +} +  struct AttrmapPass : public Pass {  	AttrmapPass() : Pass("attrmap", "renaming attributes") { }  	void help() YS_OVERRIDE @@ -151,25 +227,10 @@ struct AttrmapPass : public Pass {  		log("\n");  		log("    attrmap [options] [selection]\n");  		log("\n"); -		log("This command renames attributes and/or mapps key/value pairs to\n"); +		log("This command renames attributes and/or maps key/value pairs to\n");  		log("other key/value pairs.\n");  		log("\n"); -		log("    -tocase <name>\n"); -		log("        Match attribute names case-insensitively and set it to the specified\n"); -		log("        name.\n"); -		log("\n"); -		log("    -rename <old_name> <new_name>\n"); -		log("        Rename attributes as specified\n"); -		log("\n"); -		log("    -map <old_name>=<old_value> <new_name>=<new_value>\n"); -		log("        Map key/value pairs as indicated.\n"); -		log("\n"); -		log("    -imap <old_name>=<old_value> <new_name>=<new_value>\n"); -		log("        Like -map, but use case-insensitive match for <old_value> when\n"); -		log("        it is a string value.\n"); -		log("\n"); -		log("    -remove <name>=<value>\n"); -		log("        Remove attributes matching this pattern.\n"); +		log_attrmap_paramap_options();  		log("\n");  		log("    -modattr\n");  		log("        Operate on module attributes instead of attributes on wires and cells.\n"); @@ -190,58 +251,9 @@ struct AttrmapPass : public Pass {  		size_t argidx;  		for (argidx = 1; argidx < args.size(); argidx++)  		{ -			std::string arg = args[argidx]; -			if (arg == "-tocase" && argidx+1 < args.size()) { -				auto action = new AttrmapTocase; -				action->name = args[++argidx]; -				actions.push_back(std::unique_ptr<AttrmapAction>(action)); -				continue; -			} -			if (arg == "-rename" && argidx+2 < args.size()) { -				auto action = new AttrmapRename; -				action->old_name = args[++argidx]; -				action->new_name = args[++argidx]; -				actions.push_back(std::unique_ptr<AttrmapAction>(action)); +			if (parse_attrmap_paramap_options(argidx, args, actions))  				continue; -			} -			if ((arg == "-map" || arg == "-imap") && argidx+2 < args.size()) { -				string arg1 = args[++argidx]; -				string arg2 = args[++argidx]; -				string val1, val2; -				size_t p = arg1.find("="); -				if (p != string::npos) { -					val1 = arg1.substr(p+1); -					arg1 = arg1.substr(0, p); -				} -				p = arg2.find("="); -				if (p != string::npos) { -					val2 = arg2.substr(p+1); -					arg2 = arg2.substr(0, p); -				} -				auto action = new AttrmapMap; -				action->imap = (arg == "-map"); -				action->old_name = arg1; -				action->new_name = arg2; -				action->old_value = val1; -				action->new_value = val2; -				actions.push_back(std::unique_ptr<AttrmapAction>(action)); -				continue; -			} -			if (arg == "-remove" && argidx+1 < args.size()) { -				string arg1 = args[++argidx], val1; -				size_t p = arg1.find("="); -				if (p != string::npos) { -					val1 = arg1.substr(p+1); -					arg1 = arg1.substr(0, p); -				} -				auto action = new AttrmapRemove; -				action->name = arg1; -				action->has_value = (p != string::npos); -				action->value = val1; -				actions.push_back(std::unique_ptr<AttrmapAction>(action)); -				continue; -			} -			if (arg == "-modattr") { +			if (args[argidx] == "-modattr") {  				modattr_mode = true;  				continue;  			} @@ -287,4 +299,43 @@ struct AttrmapPass : public Pass {  	}  } AttrmapPass; +struct ParamapPass : public Pass { +	ParamapPass() : Pass("paramap", "renaming cell parameters") { } +	void help() YS_OVERRIDE +	{ +		//   |---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---| +		log("\n"); +		log("    paramap [options] [selection]\n"); +		log("\n"); +		log("This command renames cell parameters and/or maps key/value pairs to\n"); +		log("other key/value pairs.\n"); +		log("\n"); +		log_attrmap_paramap_options(); +		log("\n"); +		log("For example, mapping Diamond-style ECP5 \"init\" attributes to Yosys-style:\n"); +		log("\n"); +		log("    paramap -tocase INIT t:LUT4\n"); +		log("\n"); +	} +	void execute(std::vector<std::string> args, RTLIL::Design *design) YS_OVERRIDE +	{ +		log_header(design, "Executing PARAMAP pass (move or copy cell parameters).\n"); + +		vector<std::unique_ptr<AttrmapAction>> actions; + +		size_t argidx; +		for (argidx = 1; argidx < args.size(); argidx++) +		{ +			if (parse_attrmap_paramap_options(argidx, args, actions)) +				continue; +			break; +		} +		extra_args(args, argidx, design); + +		for (auto module : design->selected_modules()) +		for (auto cell : module->selected_cells()) +			attrmap_apply(stringf("%s.%s", log_id(module), log_id(cell)), actions, cell->parameters); +	} +} ParamapPass; +  PRIVATE_NAMESPACE_END diff --git a/passes/techmap/clkbufmap.cc b/passes/techmap/clkbufmap.cc new file mode 100644 index 000000000..246932d81 --- /dev/null +++ b/passes/techmap/clkbufmap.cc @@ -0,0 +1,298 @@ +/* + *  yosys -- Yosys Open SYnthesis Suite + * + *  Copyright (C) 2012  Clifford Wolf <clifford@clifford.at> + *  Copyright (C) 2019  Marcin Kościelnicki <mwk@0x04.net> + * + *  Permission to use, copy, modify, and/or distribute this software for any + *  purpose with or without fee is hereby granted, provided that the above + *  copyright notice and this permission notice appear in all copies. + * + *  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES + *  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF + *  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR + *  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES + *  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN + *  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF + *  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. + * + */ + +#include "kernel/yosys.h" +#include "kernel/sigtools.h" + +USING_YOSYS_NAMESPACE +PRIVATE_NAMESPACE_BEGIN + +void split_portname_pair(std::string &port1, std::string &port2) +{ +	size_t pos = port1.find_first_of(':'); +	if (pos != std::string::npos) { +		port2 = port1.substr(pos+1); +		port1 = port1.substr(0, pos); +	} +} + +struct ClkbufmapPass : public Pass { +	ClkbufmapPass() : Pass("clkbufmap", "insert global buffers on clock networks") { } +	void help() YS_OVERRIDE +	{ +		//   |---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---| +		log("\n"); +		log("    clkbufmap [options] [selection]\n"); +		log("\n"); +		log("Inserts global buffers between nets connected to clock inputs and their drivers.\n"); +		log("\n"); +		log("In the absence of any selection, all wires without the 'clkbuf_inhibit'\n"); +		log("attribute will be considered for global buffer insertion.\n"); +		log("Alternatively, to consider all wires without the 'buffer_type' attribute set to\n"); +		log("'none' or 'bufr' one would specify:\n"); +		log("  'w:* a:buffer_type=none a:buffer_type=bufr %%u %%d'\n"); +		log("as the selection.\n"); +		log("\n"); +		log("    -buf <celltype> <portname_out>:<portname_in>\n"); +		log("        Specifies the cell type to use for the global buffers\n"); +		log("        and its port names.  The first port will be connected to\n"); +		log("        the clock network sinks, and the second will be connected\n"); +		log("        to the actual clock source.  This option is required.\n"); +		log("\n"); +		log("    -inpad <celltype> <portname_out>:<portname_in>\n"); +		log("        If specified, a PAD cell of the given type is inserted on\n"); +		log("        clock nets that are also top module's inputs (in addition\n"); +		log("        to the global buffer).\n"); +		log("\n"); +	} + +	void module_queue(Design *design, Module *module, std::vector<Module *> &modules_sorted, pool<Module *> &modules_processed) { +		if (modules_processed.count(module)) +			return; +		for (auto cell : module->cells()) { +			Module *submodule = design->module(cell->type); +			if (!submodule) +				continue; +			module_queue(design, submodule, modules_sorted, modules_processed); +		} +		modules_sorted.push_back(module); +		modules_processed.insert(module); +	} + +	void execute(std::vector<std::string> args, RTLIL::Design *design) YS_OVERRIDE +	{ +		log_header(design, "Executing CLKBUFMAP pass (inserting global clock buffers).\n"); + +		std::string buf_celltype, buf_portname, buf_portname2; +		std::string inpad_celltype, inpad_portname, inpad_portname2; + +		size_t argidx; +		for (argidx = 1; argidx < args.size(); argidx++) +		{ +			std::string arg = args[argidx]; +			if (arg == "-buf" && argidx+2 < args.size()) { +				buf_celltype = args[++argidx]; +				buf_portname = args[++argidx]; +				split_portname_pair(buf_portname, buf_portname2); +				continue; +			} +			if (arg == "-inpad" && argidx+2 < args.size()) { +				inpad_celltype = args[++argidx]; +				inpad_portname = args[++argidx]; +				split_portname_pair(inpad_portname, inpad_portname2); +				continue; +			} +			break; +		} + +		bool select = false; +		if (argidx < args.size()) { +			if (args[argidx].compare(0, 1, "-") != 0) +				select = true; +			extra_args(args, argidx, design); +		} + +		if (buf_celltype.empty()) +			log_error("The -buf option is required.\n"); + +		// Cell type, port name, bit index. +		pool<pair<IdString, pair<IdString, int>>> sink_ports; +		pool<pair<IdString, pair<IdString, int>>> buf_ports; + +		// Process submodules before module using them. +		std::vector<Module *> modules_sorted; +		pool<Module *> modules_processed; +		for (auto module : design->selected_modules()) +			module_queue(design, module, modules_sorted, modules_processed); + +		for (auto module : modules_sorted) +		{ +			if (module->get_blackbox_attribute()) { +				for (auto port : module->ports) { +					auto wire = module->wire(port); +					if (wire->get_bool_attribute("\\clkbuf_driver")) +						for (int i = 0; i < GetSize(wire); i++) +							buf_ports.insert(make_pair(module->name, make_pair(wire->name, i))); +					if (wire->get_bool_attribute("\\clkbuf_sink")) +						for (int i = 0; i < GetSize(wire); i++) +							sink_ports.insert(make_pair(module->name, make_pair(wire->name, i))); +				} +				continue; +			} +			pool<SigBit> sink_wire_bits; +			pool<SigBit> buf_wire_bits; +			pool<SigBit> driven_wire_bits; +			SigMap sigmap(module); +			// bit -> (buffer, buffer's input) +			dict<SigBit, pair<Cell *, Wire *>> buffered_bits; + +			// First, collect nets that could use a clock buffer. +			for (auto cell : module->cells()) +			for (auto port : cell->connections()) +			for (int i = 0; i < port.second.size(); i++) +				if (sink_ports.count(make_pair(cell->type, make_pair(port.first, i)))) +					sink_wire_bits.insert(sigmap(port.second[i])); + +			// Second, collect ones that already have a clock buffer. +			for (auto cell : module->cells()) +			for (auto port : cell->connections()) +			for (int i = 0; i < port.second.size(); i++) +				if (buf_ports.count(make_pair(cell->type, make_pair(port.first, i)))) +					buf_wire_bits.insert(sigmap(port.second[i])); + +			// Collect all driven bits. +			for (auto cell : module->cells()) +			for (auto port : cell->connections()) +				if (cell->output(port.first)) +					for (int i = 0; i < port.second.size(); i++) +						driven_wire_bits.insert(port.second[i]); + +			// Insert buffers. +			std::vector<pair<Wire *, Wire *>> input_queue; +			// Copy current wire list, as we will be adding new ones during iteration. +			std::vector<Wire *> wires(module->wires()); +			for (auto wire : wires) +			{ +				// Should not happen. +				if (wire->port_input && wire->port_output) +					continue; +				bool process_wire = module->selected(wire); +				if (!select && wire->get_bool_attribute("\\clkbuf_inhibit")) +					process_wire = false; +				if (!process_wire) { +					// This wire is supposed to be bypassed, so make sure we don't buffer it in +					// some buffer higher up in the hierarchy. +					if (wire->port_output) +						for (int i = 0; i < GetSize(wire); i++) +							buf_ports.insert(make_pair(module->name, make_pair(wire->name, i))); +					continue; +				} + +				pool<int> input_bits; + +				for (int i = 0; i < GetSize(wire); i++) +				{ +					SigBit wire_bit(wire, i); +					SigBit mapped_wire_bit = sigmap(wire_bit); +					if (buf_wire_bits.count(mapped_wire_bit)) { +						// Already buffered downstream.  If this is an output, mark it. +						if (wire->port_output) +							buf_ports.insert(make_pair(module->name, make_pair(wire->name, i))); +					} else if (!sink_wire_bits.count(mapped_wire_bit)) { +						// Nothing to do. +					} else if (driven_wire_bits.count(wire_bit) || (wire->port_input && module->get_bool_attribute("\\top"))) { +						// Clock network not yet buffered, driven by one of +						// our cells or a top-level input -- buffer it. + +						log("Inserting %s on %s.%s[%d].\n", buf_celltype.c_str(), log_id(module), log_id(wire), i); +						RTLIL::Cell *cell = module->addCell(NEW_ID, RTLIL::escape_id(buf_celltype)); +						Wire *iwire = module->addWire(NEW_ID); +						cell->setPort(RTLIL::escape_id(buf_portname), mapped_wire_bit); +						cell->setPort(RTLIL::escape_id(buf_portname2), iwire); +						if (wire->port_input && !inpad_celltype.empty() && module->get_bool_attribute("\\top")) { +							log("Inserting %s on %s.%s[%d].\n", inpad_celltype.c_str(), log_id(module), log_id(wire), i); +							RTLIL::Cell *cell2 = module->addCell(NEW_ID, RTLIL::escape_id(inpad_celltype)); +							cell2->setPort(RTLIL::escape_id(inpad_portname), iwire); +							iwire = module->addWire(NEW_ID); +							cell2->setPort(RTLIL::escape_id(inpad_portname2), iwire); +						} +						buffered_bits[mapped_wire_bit] = make_pair(cell, iwire); + +						if (wire->port_input) { +							input_bits.insert(i); +						} +					} else if (wire->port_input) { +						// A clock input in a submodule -- mark it, let higher level +						// worry about it. +						if (wire->port_input) +							sink_ports.insert(make_pair(module->name, make_pair(wire->name, i))); +					} +				} +				if (!input_bits.empty()) { +					// This is an input port and some buffers were inserted -- we need +					// to create a new input wire and transfer attributes. +					Wire *new_wire = module->addWire(NEW_ID, wire); + +					for (int i = 0; i < wire->width; i++) { +						SigBit wire_bit(wire, i); +						SigBit mapped_wire_bit = sigmap(wire_bit); +						auto it = buffered_bits.find(mapped_wire_bit); +						if (it != buffered_bits.end()) { + +							module->connect(it->second.second, SigSpec(new_wire, i)); +						} else { +							module->connect(SigSpec(wire, i), SigSpec(new_wire, i)); +						} +					} +					input_queue.push_back(make_pair(wire, new_wire)); +				} +			} + +			// Mark any newly-buffered output ports as such. +			for (auto wire : module->selected_wires()) { +				if (wire->port_input || !wire->port_output) +					continue; +				for (int i = 0; i < GetSize(wire); i++) +				{ +					SigBit wire_bit(wire, i); +					SigBit mapped_wire_bit = sigmap(wire_bit); +					if (buffered_bits.count(mapped_wire_bit)) +						buf_ports.insert(make_pair(module->name, make_pair(wire->name, i))); +				} +			} + +			// Reconnect the drivers to buffer inputs. +			for (auto cell : module->cells()) +			for (auto port : cell->connections()) { +				if (!cell->output(port.first)) +					continue; +				SigSpec sig = port.second; +				bool newsig = false; +				for (auto &bit : sig) { +					const auto it = buffered_bits.find(sigmap(bit)); +					if (it == buffered_bits.end()) +						continue; +					// Avoid substituting buffer's own output pin. +					if (cell == it->second.first) +						continue; +					bit = it->second.second; +					newsig = true; +				} +				if (newsig) +					cell->setPort(port.first, sig); +			} + +			// This has to be done last, to avoid upsetting sigmap before the port reconnections. +			for (auto &it : input_queue) { +				Wire *wire = it.first; +				Wire *new_wire = it.second; +				module->swap_names(new_wire, wire); +				wire->attributes.clear(); +				wire->port_id = 0; +				wire->port_input = false; +				wire->port_output = false; +			} + +			module->fixup_ports(); +		} +	} +} ClkbufmapPass; + +PRIVATE_NAMESPACE_END diff --git a/passes/techmap/deminout.cc b/passes/techmap/deminout.cc index 142d12bdc..b976b401b 100644 --- a/passes/techmap/deminout.cc +++ b/passes/techmap/deminout.cc @@ -83,13 +83,13 @@ struct DeminoutPass : public Pass {  						for (auto bit : sigmap(conn.second))  							bits_used.insert(bit); -					if (conn.first == "\\Y" && cell->type.in("$mux", "$pmux", "$_MUX_", "$_TBUF_", "$tribuf")) +					if (conn.first == ID::Y && cell->type.in(ID($mux), ID($pmux), ID($_MUX_), ID($_TBUF_), ID($tribuf)))  					{ -						bool tribuf = cell->type.in("$_TBUF_", "$tribuf"); +						bool tribuf = cell->type.in(ID($_TBUF_), ID($tribuf));  						if (!tribuf) {  							for (auto &c : cell->connections()) { -								if (!c.first.in("\\A", "\\B")) +								if (!c.first.in(ID::A, ID::B))  									continue;  								for (auto b : sigmap(c.second))  									if (b == State::Sz) diff --git a/passes/techmap/dff2dffe.cc b/passes/techmap/dff2dffe.cc index c0bf3a665..0242256e5 100644 --- a/passes/techmap/dff2dffe.cc +++ b/passes/techmap/dff2dffe.cc @@ -52,13 +52,13 @@ struct Dff2dffeWorker  		}  		for (auto cell : module->cells()) { -			if (cell->type.in("$mux", "$pmux", "$_MUX_")) { -				RTLIL::SigSpec sig_y = sigmap(cell->getPort("\\Y")); +			if (cell->type.in(ID($mux), ID($pmux), ID($_MUX_))) { +				RTLIL::SigSpec sig_y = sigmap(cell->getPort(ID::Y));  				for (int i = 0; i < GetSize(sig_y); i++)  					bit2mux[sig_y[i]] = cell_int_t(cell, i);  			}  			if (direct_dict.empty()) { -				if (cell->type.in("$dff", "$_DFF_N_", "$_DFF_P_")) +				if (cell->type.in(ID($dff), ID($_DFF_N_), ID($_DFF_P_)))  					dff_cells.push_back(cell);  			} else {  				if (direct_dict.count(cell->type)) @@ -86,9 +86,9 @@ struct Dff2dffeWorker  			return ret;  		cell_int_t mux_cell_int = bit2mux.at(d); -		RTLIL::SigSpec sig_a = sigmap(mux_cell_int.first->getPort("\\A")); -		RTLIL::SigSpec sig_b = sigmap(mux_cell_int.first->getPort("\\B")); -		RTLIL::SigSpec sig_s = sigmap(mux_cell_int.first->getPort("\\S")); +		RTLIL::SigSpec sig_a = sigmap(mux_cell_int.first->getPort(ID::A)); +		RTLIL::SigSpec sig_b = sigmap(mux_cell_int.first->getPort(ID::B)); +		RTLIL::SigSpec sig_s = sigmap(mux_cell_int.first->getPort(ID(S)));  		int width = GetSize(sig_a), index = mux_cell_int.second;  		for (int i = 0; i < GetSize(sig_s); i++) @@ -97,9 +97,9 @@ struct Dff2dffeWorker  				ret = find_muxtree_feedback_patterns(sig_b[i*width + index], q, path);  				if (sig_b[i*width + index] == q) { -					RTLIL::SigSpec s = mux_cell_int.first->getPort("\\B"); +					RTLIL::SigSpec s = mux_cell_int.first->getPort(ID::B);  					s[i*width + index] = RTLIL::Sx; -					mux_cell_int.first->setPort("\\B", s); +					mux_cell_int.first->setPort(ID::B, s);  				}  				return ret; @@ -120,9 +120,9 @@ struct Dff2dffeWorker  				ret.insert(pat);  			if (sig_b[i*width + index] == q) { -				RTLIL::SigSpec s = mux_cell_int.first->getPort("\\B"); +				RTLIL::SigSpec s = mux_cell_int.first->getPort(ID::B);  				s[i*width + index] = RTLIL::Sx; -				mux_cell_int.first->setPort("\\B", s); +				mux_cell_int.first->setPort(ID::B, s);  			}  		} @@ -130,9 +130,9 @@ struct Dff2dffeWorker  			ret.insert(pat);  		if (sig_a[index] == q) { -			RTLIL::SigSpec s = mux_cell_int.first->getPort("\\A"); +			RTLIL::SigSpec s = mux_cell_int.first->getPort(ID::A);  			s[index] = RTLIL::Sx; -			mux_cell_int.first->setPort("\\A", s); +			mux_cell_int.first->setPort(ID::A, s);  		}  		return ret; @@ -185,8 +185,8 @@ struct Dff2dffeWorker  	void handle_dff_cell(RTLIL::Cell *dff_cell)  	{ -		RTLIL::SigSpec sig_d = sigmap(dff_cell->getPort("\\D")); -		RTLIL::SigSpec sig_q = sigmap(dff_cell->getPort("\\Q")); +		RTLIL::SigSpec sig_d = sigmap(dff_cell->getPort(ID(D))); +		RTLIL::SigSpec sig_q = sigmap(dff_cell->getPort(ID(Q)));  		std::map<patterns_t, std::set<int>> grouped_patterns;  		std::set<int> remaining_indices; @@ -208,16 +208,16 @@ struct Dff2dffeWorker  			}  			if (!direct_dict.empty()) {  				log("  converting %s cell %s to %s for %s -> %s.\n", log_id(dff_cell->type), log_id(dff_cell), log_id(direct_dict.at(dff_cell->type)), log_signal(new_sig_d), log_signal(new_sig_q)); -				dff_cell->setPort("\\E", make_patterns_logic(it.first, true)); +				dff_cell->setPort(ID(E), make_patterns_logic(it.first, true));  				dff_cell->type = direct_dict.at(dff_cell->type);  			} else -			if (dff_cell->type == "$dff") { -				RTLIL::Cell *new_cell = module->addDffe(NEW_ID, dff_cell->getPort("\\CLK"), make_patterns_logic(it.first, false), -						new_sig_d, new_sig_q, dff_cell->getParam("\\CLK_POLARITY").as_bool(), true); +			if (dff_cell->type == ID($dff)) { +				RTLIL::Cell *new_cell = module->addDffe(NEW_ID, dff_cell->getPort(ID(CLK)), make_patterns_logic(it.first, false), +						new_sig_d, new_sig_q, dff_cell->getParam(ID(CLK_POLARITY)).as_bool(), true);  				log("  created $dffe cell %s for %s -> %s.\n", log_id(new_cell), log_signal(new_sig_d), log_signal(new_sig_q));  			} else { -				RTLIL::Cell *new_cell = module->addDffeGate(NEW_ID, dff_cell->getPort("\\C"), make_patterns_logic(it.first, true), -						new_sig_d, new_sig_q, dff_cell->type == "$_DFF_P_", true); +				RTLIL::Cell *new_cell = module->addDffeGate(NEW_ID, dff_cell->getPort(ID(C)), make_patterns_logic(it.first, true), +						new_sig_d, new_sig_q, dff_cell->type == ID($_DFF_P_), true);  				log("  created %s cell %s for %s -> %s.\n", log_id(new_cell->type), log_id(new_cell), log_signal(new_sig_d), log_signal(new_sig_q));  			}  		} @@ -235,9 +235,9 @@ struct Dff2dffeWorker  				new_sig_d.append(sig_d[i]);  				new_sig_q.append(sig_q[i]);  			} -			dff_cell->setPort("\\D", new_sig_d); -			dff_cell->setPort("\\Q", new_sig_q); -			dff_cell->setParam("\\WIDTH", GetSize(remaining_indices)); +			dff_cell->setPort(ID(D), new_sig_d); +			dff_cell->setPort(ID(Q), new_sig_q); +			dff_cell->setParam(ID(WIDTH), GetSize(remaining_indices));  		}  	} @@ -265,7 +265,7 @@ struct Dff2dffePass : public Pass {  		log("\n");  		log("    -unmap\n");  		log("        operate in the opposite direction: replace $dffe cells with combinations\n"); -		log("        of $dff and $mux cells. the options below are ignore in unmap mode.\n"); +		log("        of $dff and $mux cells. the options below are ignored in unmap mode.\n");  		log("\n");  		log("    -unmap-mince N\n");  		log("        Same as -unmap but only unmap $dffe where the clock enable port\n"); @@ -316,25 +316,25 @@ struct Dff2dffePass : public Pass {  			if (args[argidx] == "-direct-match" && argidx + 1 < args.size()) {  				bool found_match = false;  				const char *pattern = args[++argidx].c_str(); -				if (patmatch(pattern, "$_DFF_P_"  )) found_match = true, direct_dict["$_DFF_P_"  ] = "$_DFFE_PP_"; -				if (patmatch(pattern, "$_DFF_N_"  )) found_match = true, direct_dict["$_DFF_N_"  ] = "$_DFFE_NP_"; -				if (patmatch(pattern, "$_DFF_NN0_")) found_match = true, direct_dict["$_DFF_NN0_"] = "$__DFFE_NN0"; -				if (patmatch(pattern, "$_DFF_NN1_")) found_match = true, direct_dict["$_DFF_NN1_"] = "$__DFFE_NN1"; -				if (patmatch(pattern, "$_DFF_NP0_")) found_match = true, direct_dict["$_DFF_NP0_"] = "$__DFFE_NP0"; -				if (patmatch(pattern, "$_DFF_NP1_")) found_match = true, direct_dict["$_DFF_NP1_"] = "$__DFFE_NP1"; -				if (patmatch(pattern, "$_DFF_PN0_")) found_match = true, direct_dict["$_DFF_PN0_"] = "$__DFFE_PN0"; -				if (patmatch(pattern, "$_DFF_PN1_")) found_match = true, direct_dict["$_DFF_PN1_"] = "$__DFFE_PN1"; -				if (patmatch(pattern, "$_DFF_PP0_")) found_match = true, direct_dict["$_DFF_PP0_"] = "$__DFFE_PP0"; -				if (patmatch(pattern, "$_DFF_PP1_")) found_match = true, direct_dict["$_DFF_PP1_"] = "$__DFFE_PP1"; - -				if (patmatch(pattern, "$__DFFS_NN0_")) found_match = true, direct_dict["$__DFFS_NN0_"] = "$__DFFSE_NN0"; -				if (patmatch(pattern, "$__DFFS_NN1_")) found_match = true, direct_dict["$__DFFS_NN1_"] = "$__DFFSE_NN1"; -				if (patmatch(pattern, "$__DFFS_NP0_")) found_match = true, direct_dict["$__DFFS_NP0_"] = "$__DFFSE_NP0"; -				if (patmatch(pattern, "$__DFFS_NP1_")) found_match = true, direct_dict["$__DFFS_NP1_"] = "$__DFFSE_NP1"; -				if (patmatch(pattern, "$__DFFS_PN0_")) found_match = true, direct_dict["$__DFFS_PN0_"] = "$__DFFSE_PN0"; -				if (patmatch(pattern, "$__DFFS_PN1_")) found_match = true, direct_dict["$__DFFS_PN1_"] = "$__DFFSE_PN1"; -				if (patmatch(pattern, "$__DFFS_PP0_")) found_match = true, direct_dict["$__DFFS_PP0_"] = "$__DFFSE_PP0"; -				if (patmatch(pattern, "$__DFFS_PP1_")) found_match = true, direct_dict["$__DFFS_PP1_"] = "$__DFFSE_PP1"; +				if (patmatch(pattern, "$_DFF_P_"  )) found_match = true, direct_dict[ID($_DFF_P_)  ] = ID($_DFFE_PP_); +				if (patmatch(pattern, "$_DFF_N_"  )) found_match = true, direct_dict[ID($_DFF_N_)  ] = ID($_DFFE_NP_); +				if (patmatch(pattern, "$_DFF_NN0_")) found_match = true, direct_dict[ID($_DFF_NN0_)] = ID($__DFFE_NN0); +				if (patmatch(pattern, "$_DFF_NN1_")) found_match = true, direct_dict[ID($_DFF_NN1_)] = ID($__DFFE_NN1); +				if (patmatch(pattern, "$_DFF_NP0_")) found_match = true, direct_dict[ID($_DFF_NP0_)] = ID($__DFFE_NP0); +				if (patmatch(pattern, "$_DFF_NP1_")) found_match = true, direct_dict[ID($_DFF_NP1_)] = ID($__DFFE_NP1); +				if (patmatch(pattern, "$_DFF_PN0_")) found_match = true, direct_dict[ID($_DFF_PN0_)] = ID($__DFFE_PN0); +				if (patmatch(pattern, "$_DFF_PN1_")) found_match = true, direct_dict[ID($_DFF_PN1_)] = ID($__DFFE_PN1); +				if (patmatch(pattern, "$_DFF_PP0_")) found_match = true, direct_dict[ID($_DFF_PP0_)] = ID($__DFFE_PP0); +				if (patmatch(pattern, "$_DFF_PP1_")) found_match = true, direct_dict[ID($_DFF_PP1_)] = ID($__DFFE_PP1); + +				if (patmatch(pattern, "$__DFFS_NN0_")) found_match = true, direct_dict[ID($__DFFS_NN0_)] = ID($__DFFSE_NN0); +				if (patmatch(pattern, "$__DFFS_NN1_")) found_match = true, direct_dict[ID($__DFFS_NN1_)] = ID($__DFFSE_NN1); +				if (patmatch(pattern, "$__DFFS_NP0_")) found_match = true, direct_dict[ID($__DFFS_NP0_)] = ID($__DFFSE_NP0); +				if (patmatch(pattern, "$__DFFS_NP1_")) found_match = true, direct_dict[ID($__DFFS_NP1_)] = ID($__DFFSE_NP1); +				if (patmatch(pattern, "$__DFFS_PN0_")) found_match = true, direct_dict[ID($__DFFS_PN0_)] = ID($__DFFSE_PN0); +				if (patmatch(pattern, "$__DFFS_PN1_")) found_match = true, direct_dict[ID($__DFFS_PN1_)] = ID($__DFFSE_PN1); +				if (patmatch(pattern, "$__DFFS_PP0_")) found_match = true, direct_dict[ID($__DFFS_PP0_)] = ID($__DFFSE_PP0); +				if (patmatch(pattern, "$__DFFS_PP1_")) found_match = true, direct_dict[ID($__DFFS_PP1_)] = ID($__DFFSE_PP1);  				if (!found_match)  					log_cmd_error("No cell types matched pattern '%s'.\n", pattern);  				continue; @@ -355,25 +355,25 @@ struct Dff2dffePass : public Pass {  				if (unmap_mode) {  					SigMap sigmap(mod);  					for (auto cell : mod->selected_cells()) { -						if (cell->type == "$dffe") { +						if (cell->type == ID($dffe)) {  							if (min_ce_use >= 0) {  								int ce_use = 0;  								for (auto cell_other : mod->selected_cells()) {  									if (cell_other->type != cell->type)  										continue; -									if (sigmap(cell->getPort("\\EN")) == sigmap(cell_other->getPort("\\EN"))) +									if (sigmap(cell->getPort(ID(EN))) == sigmap(cell_other->getPort(ID(EN))))  										ce_use++;  								}  								if (ce_use >= min_ce_use)  									continue;  							} -							RTLIL::SigSpec tmp = mod->addWire(NEW_ID, GetSize(cell->getPort("\\D"))); -							mod->addDff(NEW_ID, cell->getPort("\\CLK"), tmp, cell->getPort("\\Q"), cell->getParam("\\CLK_POLARITY").as_bool()); -							if (cell->getParam("\\EN_POLARITY").as_bool()) -								mod->addMux(NEW_ID, cell->getPort("\\Q"), cell->getPort("\\D"), cell->getPort("\\EN"), tmp); +							RTLIL::SigSpec tmp = mod->addWire(NEW_ID, GetSize(cell->getPort(ID(D)))); +							mod->addDff(NEW_ID, cell->getPort(ID(CLK)), tmp, cell->getPort(ID(Q)), cell->getParam(ID(CLK_POLARITY)).as_bool()); +							if (cell->getParam(ID(EN_POLARITY)).as_bool()) +								mod->addMux(NEW_ID, cell->getPort(ID(Q)), cell->getPort(ID(D)), cell->getPort(ID(EN)), tmp);  							else -								mod->addMux(NEW_ID, cell->getPort("\\D"), cell->getPort("\\Q"), cell->getPort("\\EN"), tmp); +								mod->addMux(NEW_ID, cell->getPort(ID(D)), cell->getPort(ID(Q)), cell->getPort(ID(EN)), tmp);  							mod->remove(cell);  							continue;  						} @@ -383,7 +383,7 @@ struct Dff2dffePass : public Pass {  								for (auto cell_other : mod->selected_cells()) {  									if (cell_other->type != cell->type)  										continue; -									if (sigmap(cell->getPort("\\E")) == sigmap(cell_other->getPort("\\E"))) +									if (sigmap(cell->getPort(ID(E))) == sigmap(cell_other->getPort(ID(E))))  										ce_use++;  								}  								if (ce_use >= min_ce_use) @@ -393,11 +393,11 @@ struct Dff2dffePass : public Pass {  							bool clk_pol = cell->type.compare(7, 1, "P") == 0;  							bool en_pol = cell->type.compare(8, 1, "P") == 0;  							RTLIL::SigSpec tmp = mod->addWire(NEW_ID); -							mod->addDff(NEW_ID, cell->getPort("\\C"), tmp, cell->getPort("\\Q"), clk_pol); +							mod->addDff(NEW_ID, cell->getPort(ID(C)), tmp, cell->getPort(ID(Q)), clk_pol);  							if (en_pol) -								mod->addMux(NEW_ID, cell->getPort("\\Q"), cell->getPort("\\D"), cell->getPort("\\E"), tmp); +								mod->addMux(NEW_ID, cell->getPort(ID(Q)), cell->getPort(ID(D)), cell->getPort(ID(E)), tmp);  							else -								mod->addMux(NEW_ID, cell->getPort("\\D"), cell->getPort("\\Q"), cell->getPort("\\E"), tmp); +								mod->addMux(NEW_ID, cell->getPort(ID(D)), cell->getPort(ID(Q)), cell->getPort(ID(E)), tmp);  							mod->remove(cell);  							continue;  						} diff --git a/passes/techmap/dff2dffs.cc b/passes/techmap/dff2dffs.cc index 39a4f6ade..0ea033513 100644 --- a/passes/techmap/dff2dffs.cc +++ b/passes/techmap/dff2dffs.cc @@ -51,8 +51,8 @@ struct Dff2dffsPass : public Pass {  		extra_args(args, argidx, design);  		pool<IdString> dff_types; -		dff_types.insert("$_DFF_N_"); -		dff_types.insert("$_DFF_P_"); +		dff_types.insert(ID($_DFF_N_)); +		dff_types.insert(ID($_DFF_P_));  		for (auto module : design->selected_modules())  		{ @@ -69,19 +69,19 @@ struct Dff2dffsPass : public Pass {  					continue;  				} -				if (cell->type != "$_MUX_") +				if (cell->type != ID($_MUX_))  					continue; -				SigBit bit_a = sigmap(cell->getPort("\\A")); -				SigBit bit_b = sigmap(cell->getPort("\\B")); +				SigBit bit_a = sigmap(cell->getPort(ID::A)); +				SigBit bit_b = sigmap(cell->getPort(ID::B));  				if (bit_a.wire == nullptr || bit_b.wire == nullptr) -					sr_muxes[sigmap(cell->getPort("\\Y"))] = cell; +					sr_muxes[sigmap(cell->getPort(ID::Y))] = cell;  			}  			for (auto cell : ff_cells)  			{ -				SigSpec sig_d = cell->getPort("\\D"); +				SigSpec sig_d = cell->getPort(ID(D));  				if (GetSize(sig_d) < 1)  					continue; @@ -92,9 +92,9 @@ struct Dff2dffsPass : public Pass {  					continue;  				Cell *mux_cell = sr_muxes.at(bit_d); -				SigBit bit_a = sigmap(mux_cell->getPort("\\A")); -				SigBit bit_b = sigmap(mux_cell->getPort("\\B")); -				SigBit bit_s = sigmap(mux_cell->getPort("\\S")); +				SigBit bit_a = sigmap(mux_cell->getPort(ID::A)); +				SigBit bit_b = sigmap(mux_cell->getPort(ID::B)); +				SigBit bit_s = sigmap(mux_cell->getPort(ID(S)));  				log("  Merging %s (A=%s, B=%s, S=%s) into %s (%s).\n", log_id(mux_cell),  						log_signal(bit_a), log_signal(bit_b), log_signal(bit_s), log_id(cell), log_id(cell->type)); @@ -114,26 +114,26 @@ struct Dff2dffsPass : public Pass {  				}  				if (sr_val == State::S1) { -					if (cell->type == "$_DFF_N_") { -						if (invert_sr) cell->type = "$__DFFS_NN1_"; -						else cell->type = "$__DFFS_NP1_"; +					if (cell->type == ID($_DFF_N_)) { +						if (invert_sr) cell->type = ID($__DFFS_NN1_); +						else cell->type = ID($__DFFS_NP1_);  					} else { -						log_assert(cell->type == "$_DFF_P_"); -						if (invert_sr) cell->type = "$__DFFS_PN1_"; -						else cell->type = "$__DFFS_PP1_"; +						log_assert(cell->type == ID($_DFF_P_)); +						if (invert_sr) cell->type = ID($__DFFS_PN1_); +						else cell->type = ID($__DFFS_PP1_);  					}  				} else { -					if (cell->type == "$_DFF_N_") { -						if (invert_sr) cell->type = "$__DFFS_NN0_"; -						else cell->type = "$__DFFS_NP0_"; +					if (cell->type == ID($_DFF_N_)) { +						if (invert_sr) cell->type = ID($__DFFS_NN0_); +						else cell->type = ID($__DFFS_NP0_);  					} else { -						log_assert(cell->type == "$_DFF_P_"); -						if (invert_sr) cell->type = "$__DFFS_PN0_"; -						else cell->type = "$__DFFS_PP0_"; +						log_assert(cell->type == ID($_DFF_P_)); +						if (invert_sr) cell->type = ID($__DFFS_PN0_); +						else cell->type = ID($__DFFS_PP0_);  					}  				} -				cell->setPort("\\R", sr_sig); -				cell->setPort("\\D", bit_d); +				cell->setPort(ID(R), sr_sig); +				cell->setPort(ID(D), bit_d);  			}  		}  	} diff --git a/passes/techmap/dffinit.cc b/passes/techmap/dffinit.cc index 0ad33dc0e..cf9301442 100644 --- a/passes/techmap/dffinit.cc +++ b/passes/techmap/dffinit.cc @@ -99,8 +99,8 @@ struct DffinitPass : public Pass {  			pool<SigBit> used_bits;  			for (auto wire : module->selected_wires()) { -				if (wire->attributes.count("\\init")) { -					Const value = wire->attributes.at("\\init"); +				if (wire->attributes.count(ID(init))) { +					Const value = wire->attributes.at(ID(init));  					for (int i = 0; i < min(GetSize(value), GetSize(wire)); i++)  						if (value[i] != State::Sx)  							init_bits[sigmap(SigBit(wire, i))] = value[i]; @@ -161,8 +161,8 @@ struct DffinitPass : public Pass {  			}  			for (auto wire : module->selected_wires()) -				if (wire->attributes.count("\\init")) { -					Const &value = wire->attributes.at("\\init"); +				if (wire->attributes.count(ID(init))) { +					Const &value = wire->attributes.at(ID(init));  					bool do_cleanup = true;  					for (int i = 0; i < min(GetSize(value), GetSize(wire)); i++) {  						SigBit bit = sigmap(SigBit(wire, i)); @@ -173,7 +173,7 @@ struct DffinitPass : public Pass {  					}  					if (do_cleanup) {  						log("Removing init attribute from wire %s.%s.\n", log_id(module), log_id(wire)); -						wire->attributes.erase("\\init"); +						wire->attributes.erase(ID(init));  					}  				}  		} diff --git a/passes/techmap/dfflibmap.cc b/passes/techmap/dfflibmap.cc index b5c0498d0..b15109cd3 100644 --- a/passes/techmap/dfflibmap.cc +++ b/passes/techmap/dfflibmap.cc @@ -27,12 +27,12 @@ USING_YOSYS_NAMESPACE  PRIVATE_NAMESPACE_BEGIN  struct cell_mapping { -	std::string cell_name; +	IdString cell_name;  	std::map<std::string, char> ports;  };  static std::map<RTLIL::IdString, cell_mapping> cell_mappings; -static void logmap(std::string dff) +static void logmap(IdString dff)  {  	if (cell_mappings.count(dff) == 0) {  		log("    unmapped dff cell: %s\n", dff.c_str()); @@ -54,26 +54,26 @@ static void logmap(std::string dff)  static void logmap_all()  { -	logmap("$_DFF_N_"); -	logmap("$_DFF_P_"); - -	logmap("$_DFF_NN0_"); -	logmap("$_DFF_NN1_"); -	logmap("$_DFF_NP0_"); -	logmap("$_DFF_NP1_"); -	logmap("$_DFF_PN0_"); -	logmap("$_DFF_PN1_"); -	logmap("$_DFF_PP0_"); -	logmap("$_DFF_PP1_"); - -	logmap("$_DFFSR_NNN_"); -	logmap("$_DFFSR_NNP_"); -	logmap("$_DFFSR_NPN_"); -	logmap("$_DFFSR_NPP_"); -	logmap("$_DFFSR_PNN_"); -	logmap("$_DFFSR_PNP_"); -	logmap("$_DFFSR_PPN_"); -	logmap("$_DFFSR_PPP_"); +	logmap(ID($_DFF_N_)); +	logmap(ID($_DFF_P_)); + +	logmap(ID($_DFF_NN0_)); +	logmap(ID($_DFF_NN1_)); +	logmap(ID($_DFF_NP0_)); +	logmap(ID($_DFF_NP1_)); +	logmap(ID($_DFF_PN0_)); +	logmap(ID($_DFF_PN1_)); +	logmap(ID($_DFF_PP0_)); +	logmap(ID($_DFF_PP1_)); + +	logmap(ID($_DFFSR_NNN_)); +	logmap(ID($_DFFSR_NNP_)); +	logmap(ID($_DFFSR_NPN_)); +	logmap(ID($_DFFSR_NPP_)); +	logmap(ID($_DFFSR_PNN_)); +	logmap(ID($_DFFSR_PNP_)); +	logmap(ID($_DFFSR_PPN_)); +	logmap(ID($_DFFSR_PPP_));  }  static bool parse_pin(LibertyAst *cell, LibertyAst *attr, std::string &pin_name, bool &pin_pol) @@ -115,7 +115,7 @@ static bool parse_pin(LibertyAst *cell, LibertyAst *attr, std::string &pin_name,  	return false;  } -static void find_cell(LibertyAst *ast, std::string cell_type, bool clkpol, bool has_reset, bool rstpol, bool rstval, bool prepare_mode) +static void find_cell(LibertyAst *ast, IdString cell_type, bool clkpol, bool has_reset, bool rstpol, bool rstval, bool prepare_mode)  {  	LibertyAst *best_cell = NULL;  	std::map<std::string, char> best_cell_ports; @@ -230,13 +230,13 @@ static void find_cell(LibertyAst *ast, std::string cell_type, bool clkpol, bool  			cell_mappings[cell_type].ports["D"] = 'D';  			cell_mappings[cell_type].ports["Q"] = 'Q';  		} else { -			cell_mappings[cell_type].cell_name = best_cell->args[0]; +			cell_mappings[cell_type].cell_name = RTLIL::escape_id(best_cell->args[0]);  			cell_mappings[cell_type].ports = best_cell_ports;  		}  	}  } -static void find_cell_sr(LibertyAst *ast, std::string cell_type, bool clkpol, bool setpol, bool clrpol, bool prepare_mode) +static void find_cell_sr(LibertyAst *ast, IdString cell_type, bool clkpol, bool setpol, bool clrpol, bool prepare_mode)  {  	LibertyAst *best_cell = NULL;  	std::map<std::string, char> best_cell_ports; @@ -347,7 +347,7 @@ static void find_cell_sr(LibertyAst *ast, std::string cell_type, bool clkpol, bo  			cell_mappings[cell_type].ports["D"] = 'D';  			cell_mappings[cell_type].ports["Q"] = 'Q';  		} else { -			cell_mappings[cell_type].cell_name = best_cell->args[0]; +			cell_mappings[cell_type].cell_name = RTLIL::escape_id(best_cell->args[0]);  			cell_mappings[cell_type].ports = best_cell_ports;  		}  	} @@ -404,7 +404,7 @@ static bool expand_cellmap(std::string pattern, std::string inv)  	return return_status;  } -static void map_sr_to_arst(const char *from, const char *to) +static void map_sr_to_arst(IdString from, IdString to)  {  	if (!cell_mappings.count(from) || cell_mappings.count(to) > 0)  		return; @@ -419,7 +419,7 @@ static void map_sr_to_arst(const char *from, const char *to)  	log_assert(from_clk_pol == to_clk_pol);  	log_assert(to_rst_pol == from_set_pol && to_rst_pol == from_clr_pol); -	log("  create mapping for %s from mapping for %s.\n", to, from); +	log("  create mapping for %s from mapping for %s.\n", to.c_str(), from.c_str());  	cell_mappings[to].cell_name = cell_mappings[from].cell_name;  	cell_mappings[to].ports = cell_mappings[from].ports; @@ -450,7 +450,7 @@ static void map_sr_to_arst(const char *from, const char *to)  	}  } -static void map_adff_to_dff(const char *from, const char *to) +static void map_adff_to_dff(IdString from, IdString to)  {  	if (!cell_mappings.count(from) || cell_mappings.count(to) > 0)  		return; @@ -461,7 +461,7 @@ static void map_adff_to_dff(const char *from, const char *to)  	log_assert(from_clk_pol == to_clk_pol); -	log("  create mapping for %s from mapping for %s.\n", to, from); +	log("  create mapping for %s from mapping for %s.\n", to.c_str(), from.c_str());  	cell_mappings[to].cell_name = cell_mappings[from].cell_name;  	cell_mappings[to].ports = cell_mappings[from].ports; @@ -484,8 +484,8 @@ static void dfflibmap(RTLIL::Design *design, RTLIL::Module *module, bool prepare  	for (auto &it : module->cells_) {  		if (design->selected(module, it.second) && cell_mappings.count(it.second->type) > 0)  			cell_list.push_back(it.second); -		if (it.second->type == "$_NOT_") -			notmap[sigmap(it.second->getPort("\\A"))].insert(it.second); +		if (it.second->type == ID($_NOT_)) +			notmap[sigmap(it.second->getPort(ID::A))].insert(it.second);  	}  	std::map<std::string, int> stats; @@ -499,7 +499,7 @@ static void dfflibmap(RTLIL::Design *design, RTLIL::Module *module, bool prepare  		module->remove(cell);  		cell_mapping &cm = cell_mappings[cell_type]; -		RTLIL::Cell *new_cell = module->addCell(cell_name, prepare_mode ? cm.cell_name : "\\" + cm.cell_name); +		RTLIL::Cell *new_cell = module->addCell(cell_name, prepare_mode ? cm.cell_name : cm.cell_name);  		new_cell->set_src_attribute(src); @@ -519,8 +519,8 @@ static void dfflibmap(RTLIL::Design *design, RTLIL::Module *module, bool prepare  				sig = module->addWire(NEW_ID, GetSize(old_sig));  				if (has_q && has_qn) {  					for (auto &it : notmap[sigmap(old_sig)]) { -						module->connect(it->getPort("\\Y"), sig); -						it->setPort("\\Y", module->addWire(NEW_ID, GetSize(old_sig))); +						module->connect(it->getPort(ID::Y), sig); +						it->setPort(ID::Y, module->addWire(NEW_ID, GetSize(old_sig)));  					}  				} else {  					module->addNotGate(NEW_ID, sig, old_sig); @@ -599,26 +599,26 @@ struct DfflibmapPass : public Pass {  		LibertyParser libparser(f);  		f.close(); -		find_cell(libparser.ast, "$_DFF_N_", false, false, false, false, prepare_mode); -		find_cell(libparser.ast, "$_DFF_P_", true, false, false, false, prepare_mode); - -		find_cell(libparser.ast, "$_DFF_NN0_", false, true, false, false, prepare_mode); -		find_cell(libparser.ast, "$_DFF_NN1_", false, true, false, true, prepare_mode); -		find_cell(libparser.ast, "$_DFF_NP0_", false, true, true, false, prepare_mode); -		find_cell(libparser.ast, "$_DFF_NP1_", false, true, true, true, prepare_mode); -		find_cell(libparser.ast, "$_DFF_PN0_", true, true, false, false, prepare_mode); -		find_cell(libparser.ast, "$_DFF_PN1_", true, true, false, true, prepare_mode); -		find_cell(libparser.ast, "$_DFF_PP0_", true, true, true, false, prepare_mode); -		find_cell(libparser.ast, "$_DFF_PP1_", true, true, true, true, prepare_mode); - -		find_cell_sr(libparser.ast, "$_DFFSR_NNN_", false, false, false, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_NNP_", false, false, true, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_NPN_", false, true, false, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_NPP_", false, true, true, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_PNN_", true, false, false, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_PNP_", true, false, true, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_PPN_", true, true, false, prepare_mode); -		find_cell_sr(libparser.ast, "$_DFFSR_PPP_", true, true, true, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_N_), false, false, false, false, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_P_), true, false, false, false, prepare_mode); + +		find_cell(libparser.ast, ID($_DFF_NN0_), false, true, false, false, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_NN1_), false, true, false, true, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_NP0_), false, true, true, false, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_NP1_), false, true, true, true, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_PN0_), true, true, false, false, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_PN1_), true, true, false, true, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_PP0_), true, true, true, false, prepare_mode); +		find_cell(libparser.ast, ID($_DFF_PP1_), true, true, true, true, prepare_mode); + +		find_cell_sr(libparser.ast, ID($_DFFSR_NNN_), false, false, false, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_NNP_), false, false, true, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_NPN_), false, true, false, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_NPP_), false, true, true, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_PNN_), true, false, false, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_PNP_), true, false, true, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_PPN_), true, true, false, prepare_mode); +		find_cell_sr(libparser.ast, ID($_DFFSR_PPP_), true, true, true, prepare_mode);  		// try to implement as many cells as possible just by inverting  		// the SET and RESET pins. If necessary, implement cell types @@ -642,23 +642,23 @@ struct DfflibmapPass : public Pass {  			break;  		} -		map_sr_to_arst("$_DFFSR_NNN_", "$_DFF_NN0_"); -		map_sr_to_arst("$_DFFSR_NNN_", "$_DFF_NN1_"); -		map_sr_to_arst("$_DFFSR_NPP_", "$_DFF_NP0_"); -		map_sr_to_arst("$_DFFSR_NPP_", "$_DFF_NP1_"); -		map_sr_to_arst("$_DFFSR_PNN_", "$_DFF_PN0_"); -		map_sr_to_arst("$_DFFSR_PNN_", "$_DFF_PN1_"); -		map_sr_to_arst("$_DFFSR_PPP_", "$_DFF_PP0_"); -		map_sr_to_arst("$_DFFSR_PPP_", "$_DFF_PP1_"); - -		map_adff_to_dff("$_DFF_NN0_", "$_DFF_N_"); -		map_adff_to_dff("$_DFF_NN1_", "$_DFF_N_"); -		map_adff_to_dff("$_DFF_NP0_", "$_DFF_N_"); -		map_adff_to_dff("$_DFF_NP1_", "$_DFF_N_"); -		map_adff_to_dff("$_DFF_PN0_", "$_DFF_P_"); -		map_adff_to_dff("$_DFF_PN1_", "$_DFF_P_"); -		map_adff_to_dff("$_DFF_PP0_", "$_DFF_P_"); -		map_adff_to_dff("$_DFF_PP1_", "$_DFF_P_"); +		map_sr_to_arst(ID($_DFFSR_NNN_), ID($_DFF_NN0_)); +		map_sr_to_arst(ID($_DFFSR_NNN_), ID($_DFF_NN1_)); +		map_sr_to_arst(ID($_DFFSR_NPP_), ID($_DFF_NP0_)); +		map_sr_to_arst(ID($_DFFSR_NPP_), ID($_DFF_NP1_)); +		map_sr_to_arst(ID($_DFFSR_PNN_), ID($_DFF_PN0_)); +		map_sr_to_arst(ID($_DFFSR_PNN_), ID($_DFF_PN1_)); +		map_sr_to_arst(ID($_DFFSR_PPP_), ID($_DFF_PP0_)); +		map_sr_to_arst(ID($_DFFSR_PPP_), ID($_DFF_PP1_)); + +		map_adff_to_dff(ID($_DFF_NN0_), ID($_DFF_N_)); +		map_adff_to_dff(ID($_DFF_NN1_), ID($_DFF_N_)); +		map_adff_to_dff(ID($_DFF_NP0_), ID($_DFF_N_)); +		map_adff_to_dff(ID($_DFF_NP1_), ID($_DFF_N_)); +		map_adff_to_dff(ID($_DFF_PN0_), ID($_DFF_P_)); +		map_adff_to_dff(ID($_DFF_PN1_), ID($_DFF_P_)); +		map_adff_to_dff(ID($_DFF_PP0_), ID($_DFF_P_)); +		map_adff_to_dff(ID($_DFF_PP1_), ID($_DFF_P_));  		log("  final dff cell mappings:\n");  		logmap_all(); diff --git a/passes/techmap/dffsr2dff.cc b/passes/techmap/dffsr2dff.cc index 086a1d2fa..61b06fdc1 100644 --- a/passes/techmap/dffsr2dff.cc +++ b/passes/techmap/dffsr2dff.cc @@ -25,17 +25,17 @@ PRIVATE_NAMESPACE_BEGIN  void dffsr_worker(SigMap &sigmap, Module *module, Cell *cell)  { -	if (cell->type == "$dffsr") +	if (cell->type == ID($dffsr))  	{ -		int width = cell->getParam("\\WIDTH").as_int(); -		bool setpol = cell->getParam("\\SET_POLARITY").as_bool(); -		bool clrpol = cell->getParam("\\CLR_POLARITY").as_bool(); +		int width = cell->getParam(ID(WIDTH)).as_int(); +		bool setpol = cell->getParam(ID(SET_POLARITY)).as_bool(); +		bool clrpol = cell->getParam(ID(CLR_POLARITY)).as_bool();  		SigBit setunused = setpol ? State::S0 : State::S1;  		SigBit clrunused = clrpol ? State::S0 : State::S1; -		SigSpec setsig = sigmap(cell->getPort("\\SET")); -		SigSpec clrsig = sigmap(cell->getPort("\\CLR")); +		SigSpec setsig = sigmap(cell->getPort(ID(SET))); +		SigSpec clrsig = sigmap(cell->getPort(ID(CLR)));  		Const reset_val;  		SigSpec setctrl, clrctrl; @@ -78,32 +78,32 @@ void dffsr_worker(SigMap &sigmap, Module *module, Cell *cell)  		log("Converting %s cell %s.%s to $adff.\n", log_id(cell->type), log_id(module), log_id(cell));  		if (GetSize(setctrl) == 1) { -			cell->setPort("\\ARST", setctrl); -			cell->setParam("\\ARST_POLARITY", setpol); +			cell->setPort(ID(ARST), setctrl); +			cell->setParam(ID(ARST_POLARITY), setpol);  		} else { -			cell->setPort("\\ARST", clrctrl); -			cell->setParam("\\ARST_POLARITY", clrpol); +			cell->setPort(ID(ARST), clrctrl); +			cell->setParam(ID(ARST_POLARITY), clrpol);  		} -		cell->type = "$adff"; -		cell->unsetPort("\\SET"); -		cell->unsetPort("\\CLR"); -		cell->setParam("\\ARST_VALUE", reset_val); -		cell->unsetParam("\\SET_POLARITY"); -		cell->unsetParam("\\CLR_POLARITY"); +		cell->type = ID($adff); +		cell->unsetPort(ID(SET)); +		cell->unsetPort(ID(CLR)); +		cell->setParam(ID(ARST_VALUE), reset_val); +		cell->unsetParam(ID(SET_POLARITY)); +		cell->unsetParam(ID(CLR_POLARITY));  		return;  	} -	if (cell->type.in("$_DFFSR_NNN_", "$_DFFSR_NNP_", "$_DFFSR_NPN_", "$_DFFSR_NPP_", -			"$_DFFSR_PNN_", "$_DFFSR_PNP_", "$_DFFSR_PPN_", "$_DFFSR_PPP_")) +	if (cell->type.in(ID($_DFFSR_NNN_), ID($_DFFSR_NNP_), ID($_DFFSR_NPN_), ID($_DFFSR_NPP_), +			ID($_DFFSR_PNN_), ID($_DFFSR_PNP_), ID($_DFFSR_PPN_), ID($_DFFSR_PPP_)))  	{  		char clkpol = cell->type.c_str()[8];  		char setpol = cell->type.c_str()[9];  		char clrpol = cell->type.c_str()[10]; -		SigBit setbit = sigmap(cell->getPort("\\S")); -		SigBit clrbit = sigmap(cell->getPort("\\R")); +		SigBit setbit = sigmap(cell->getPort(ID(S))); +		SigBit clrbit = sigmap(cell->getPort(ID(R)));  		SigBit setunused = setpol == 'P' ? State::S0 : State::S1;  		SigBit clrunused = clrpol == 'P' ? State::S0 : State::S1; @@ -112,14 +112,14 @@ void dffsr_worker(SigMap &sigmap, Module *module, Cell *cell)  		if (setbit == setunused) {  			cell->type = stringf("$_DFF_%c%c0_", clkpol, clrpol); -			cell->unsetPort("\\S"); +			cell->unsetPort(ID(S));  			goto converted_gate;  		}  		if (clrbit == clrunused) {  			cell->type = stringf("$_DFF_%c%c1_", clkpol, setpol); -			cell->setPort("\\R", cell->getPort("\\S")); -			cell->unsetPort("\\S"); +			cell->setPort(ID(R), cell->getPort(ID(S))); +			cell->unsetPort(ID(S));  			goto converted_gate;  		} @@ -133,32 +133,32 @@ void dffsr_worker(SigMap &sigmap, Module *module, Cell *cell)  void adff_worker(SigMap &sigmap, Module *module, Cell *cell)  { -	if (cell->type == "$adff") +	if (cell->type == ID($adff))  	{ -		bool rstpol = cell->getParam("\\ARST_POLARITY").as_bool(); +		bool rstpol = cell->getParam(ID(ARST_POLARITY)).as_bool();  		SigBit rstunused = rstpol ? State::S0 : State::S1; -		SigSpec rstsig = sigmap(cell->getPort("\\ARST")); +		SigSpec rstsig = sigmap(cell->getPort(ID(ARST)));  		if (rstsig != rstunused)  			return;  		log("Converting %s cell %s.%s to $dff.\n", log_id(cell->type), log_id(module), log_id(cell)); -		cell->type = "$dff"; -		cell->unsetPort("\\ARST"); -		cell->unsetParam("\\ARST_VALUE"); -		cell->unsetParam("\\ARST_POLARITY"); +		cell->type = ID($dff); +		cell->unsetPort(ID(ARST)); +		cell->unsetParam(ID(ARST_VALUE)); +		cell->unsetParam(ID(ARST_POLARITY));  		return;  	} -	if (cell->type.in("$_DFF_NN0_", "$_DFF_NN1_", "$_DFF_NP0_", "$_DFF_NP1_", -			"$_DFF_PN0_", "$_DFF_PN1_", "$_DFF_PP0_", "$_DFF_PP1_")) +	if (cell->type.in(ID($_DFF_NN0_), ID($_DFF_NN1_), ID($_DFF_NP0_), ID($_DFF_NP1_), +			ID($_DFF_PN0_), ID($_DFF_PN1_), ID($_DFF_PP0_), ID($_DFF_PP1_)))  	{  		char clkpol = cell->type.c_str()[6];  		char rstpol = cell->type.c_str()[7]; -		SigBit rstbit = sigmap(cell->getPort("\\R")); +		SigBit rstbit = sigmap(cell->getPort(ID(R)));  		SigBit rstunused = rstpol == 'P' ? State::S0 : State::S1;  		if (rstbit != rstunused) @@ -168,7 +168,7 @@ void adff_worker(SigMap &sigmap, Module *module, Cell *cell)  		log("Converting %s cell %s.%s to %s.\n", log_id(cell->type), log_id(module), log_id(cell), log_id(newtype));  		cell->type = newtype; -		cell->unsetPort("\\R"); +		cell->unsetPort(ID(R));  		return;  	} diff --git a/passes/techmap/extract.cc b/passes/techmap/extract.cc index e3b4faba1..f8798eea5 100644 --- a/passes/techmap/extract.cc +++ b/passes/techmap/extract.cc @@ -58,36 +58,36 @@ public:  			return value;  	#define param_bool(_n) if (param == _n) return value.as_bool(); -		param_bool("\\ARST_POLARITY"); -		param_bool("\\A_SIGNED"); -		param_bool("\\B_SIGNED"); -		param_bool("\\CLK_ENABLE"); -		param_bool("\\CLK_POLARITY"); -		param_bool("\\CLR_POLARITY"); -		param_bool("\\EN_POLARITY"); -		param_bool("\\SET_POLARITY"); -		param_bool("\\TRANSPARENT"); +		param_bool(ID(ARST_POLARITY)); +		param_bool(ID(A_SIGNED)); +		param_bool(ID(B_SIGNED)); +		param_bool(ID(CLK_ENABLE)); +		param_bool(ID(CLK_POLARITY)); +		param_bool(ID(CLR_POLARITY)); +		param_bool(ID(EN_POLARITY)); +		param_bool(ID(SET_POLARITY)); +		param_bool(ID(TRANSPARENT));  	#undef param_bool  	#define param_int(_n) if (param == _n) return value.as_int(); -		param_int("\\ABITS") -		param_int("\\A_WIDTH") -		param_int("\\B_WIDTH") -		param_int("\\CTRL_IN_WIDTH") -		param_int("\\CTRL_OUT_WIDTH") -		param_int("\\OFFSET") -		param_int("\\PRIORITY") -		param_int("\\RD_PORTS") -		param_int("\\SIZE") -		param_int("\\STATE_BITS") -		param_int("\\STATE_NUM") -		param_int("\\STATE_NUM_LOG2") -		param_int("\\STATE_RST") -		param_int("\\S_WIDTH") -		param_int("\\TRANS_NUM") -		param_int("\\WIDTH") -		param_int("\\WR_PORTS") -		param_int("\\Y_WIDTH") +		param_int(ID(ABITS)) +		param_int(ID(A_WIDTH)) +		param_int(ID(B_WIDTH)) +		param_int(ID(CTRL_IN_WIDTH)) +		param_int(ID(CTRL_OUT_WIDTH)) +		param_int(ID(OFFSET)) +		param_int(ID(PRIORITY)) +		param_int(ID(RD_PORTS)) +		param_int(ID(SIZE)) +		param_int(ID(STATE_BITS)) +		param_int(ID(STATE_NUM)) +		param_int(ID(STATE_NUM_LOG2)) +		param_int(ID(STATE_RST)) +		param_int(ID(S_WIDTH)) +		param_int(ID(TRANS_NUM)) +		param_int(ID(WIDTH)) +		param_int(ID(WR_PORTS)) +		param_int(ID(Y_WIDTH))  	#undef param_int  		return value; @@ -341,10 +341,10 @@ RTLIL::Cell *replace(RTLIL::Module *needle, RTLIL::Module *haystack, SubCircuit:  bool compareSortNeedleList(RTLIL::Module *left, RTLIL::Module *right)  {  	int left_idx = 0, right_idx = 0; -	if (left->attributes.count("\\extract_order") > 0) -		left_idx = left->attributes.at("\\extract_order").as_int(); -	if (right->attributes.count("\\extract_order") > 0) -		right_idx = right->attributes.at("\\extract_order").as_int(); +	if (left->attributes.count(ID(extract_order)) > 0) +		left_idx = left->attributes.at(ID(extract_order)).as_int(); +	if (right->attributes.count(ID(extract_order)) > 0) +		right_idx = right->attributes.at(ID(extract_order)).as_int();  	if (left_idx != right_idx)  		return left_idx < right_idx;  	return left->name < right->name; diff --git a/passes/techmap/extract_counter.cc b/passes/techmap/extract_counter.cc index a8d0bc834..17a99493d 100644 --- a/passes/techmap/extract_counter.cc +++ b/passes/techmap/extract_counter.cc @@ -120,71 +120,71 @@ int counter_tryextract(  	//A counter with less than 2 bits makes no sense  	//TODO: configurable min threshold -	int a_width = cell->getParam("\\A_WIDTH").as_int(); +	int a_width = cell->getParam(ID(A_WIDTH)).as_int();  	extract.width = a_width;  	if( (a_width < 2) || (a_width > maxwidth) )  		return 1;  	//Second input must be a single bit -	int b_width = cell->getParam("\\B_WIDTH").as_int(); +	int b_width = cell->getParam(ID(B_WIDTH)).as_int();  	if(b_width != 1)  		return 2;  	//Both inputs must be unsigned, so don't extract anything with a signed input -	bool a_sign = cell->getParam("\\A_SIGNED").as_bool(); -	bool b_sign = cell->getParam("\\B_SIGNED").as_bool(); +	bool a_sign = cell->getParam(ID(A_SIGNED)).as_bool(); +	bool b_sign = cell->getParam(ID(B_SIGNED)).as_bool();  	if(a_sign || b_sign)  		return 3;  	//To be a counter, one input of the ALU must be a constant 1  	//TODO: can A or B be swapped in synthesized RTL or is B always the 1? -	const RTLIL::SigSpec b_port = sigmap(cell->getPort("\\B")); +	const RTLIL::SigSpec b_port = sigmap(cell->getPort(ID::B));  	if(!b_port.is_fully_const() || (b_port.as_int() != 1) )  		return 4;  	//BI and CI must be constant 1 as well -	const RTLIL::SigSpec bi_port = sigmap(cell->getPort("\\BI")); +	const RTLIL::SigSpec bi_port = sigmap(cell->getPort(ID(BI)));  	if(!bi_port.is_fully_const() || (bi_port.as_int() != 1) )  		return 5; -	const RTLIL::SigSpec ci_port = sigmap(cell->getPort("\\CI")); +	const RTLIL::SigSpec ci_port = sigmap(cell->getPort(ID(CI)));  	if(!ci_port.is_fully_const() || (ci_port.as_int() != 1) )  		return 6;  	//CO and X must be unconnected (exactly one connection to each port) -	if(!is_unconnected(sigmap(cell->getPort("\\CO")), index)) +	if(!is_unconnected(sigmap(cell->getPort(ID(CO))), index))  		return 7; -	if(!is_unconnected(sigmap(cell->getPort("\\X")), index)) +	if(!is_unconnected(sigmap(cell->getPort(ID(X))), index))  		return 8;  	//Y must have exactly one connection, and it has to be a $mux cell.  	//We must have a direct bus connection from our Y to their A. -	const RTLIL::SigSpec aluy = sigmap(cell->getPort("\\Y")); +	const RTLIL::SigSpec aluy = sigmap(cell->getPort(ID::Y));  	pool<Cell*> y_loads = get_other_cells(aluy, index, cell);  	if(y_loads.size() != 1)  		return 9;  	Cell* count_mux = *y_loads.begin();  	extract.count_mux = count_mux; -	if(count_mux->type != "$mux") +	if(count_mux->type != ID($mux))  		return 10; -	if(!is_full_bus(aluy, index, cell, "\\Y", count_mux, "\\A")) +	if(!is_full_bus(aluy, index, cell, ID::Y, count_mux, ID::A))  		return 11;  	//B connection of the mux is our underflow value -	const RTLIL::SigSpec underflow = sigmap(count_mux->getPort("\\B")); +	const RTLIL::SigSpec underflow = sigmap(count_mux->getPort(ID::B));  	if(!underflow.is_fully_const())  		return 12;  	extract.count_value = underflow.as_int();  	//S connection of the mux must come from an inverter (need not be the only load) -	const RTLIL::SigSpec muxsel = sigmap(count_mux->getPort("\\S")); +	const RTLIL::SigSpec muxsel = sigmap(count_mux->getPort(ID(S)));  	extract.outsig = muxsel;  	pool<Cell*> muxsel_conns = get_other_cells(muxsel, index, count_mux);  	Cell* underflow_inv = NULL;  	for(auto c : muxsel_conns)  	{ -		if(c->type != "$logic_not") +		if(c->type != ID($logic_not))  			continue; -		if(!is_full_bus(muxsel, index, c, "\\Y", count_mux, "\\S", true)) +		if(!is_full_bus(muxsel, index, c, ID::Y, count_mux, ID(S), true))  			continue;  		underflow_inv = c; @@ -196,7 +196,7 @@ int counter_tryextract(  	//Y connection of the mux must have exactly one load, the counter's internal register, if there's no clock enable  	//If we have a clock enable, Y drives the B input of a mux. A of that mux must come from our register -	const RTLIL::SigSpec muxy = sigmap(count_mux->getPort("\\Y")); +	const RTLIL::SigSpec muxy = sigmap(count_mux->getPort(ID::Y));  	pool<Cell*> muxy_loads = get_other_cells(muxy, index, count_mux);  	if(muxy_loads.size() != 1)  		return 14; @@ -204,12 +204,12 @@ int counter_tryextract(  	Cell* count_reg = muxload;  	Cell* cemux = NULL;  	RTLIL::SigSpec cey; -	if(muxload->type == "$mux") +	if(muxload->type == ID($mux))  	{  		//This mux is probably a clock enable mux.  		//Find our count register (should be our only load)  		cemux = muxload; -		cey = sigmap(cemux->getPort("\\Y")); +		cey = sigmap(cemux->getPort(ID::Y));  		pool<Cell*> cey_loads = get_other_cells(cey, index, cemux);  		if(cey_loads.size() != 1)  			return 24; @@ -217,32 +217,32 @@ int counter_tryextract(  		//Mux should have A driven by count Q, and B by muxy  		//TODO: if A and B are swapped, CE polarity is inverted -		if(sigmap(cemux->getPort("\\B")) != muxy) +		if(sigmap(cemux->getPort(ID::B)) != muxy)  			return 24; -		if(sigmap(cemux->getPort("\\A")) != sigmap(count_reg->getPort("\\Q"))) +		if(sigmap(cemux->getPort(ID::A)) != sigmap(count_reg->getPort(ID(Q))))  			return 24; -		if(sigmap(cemux->getPort("\\Y")) != sigmap(count_reg->getPort("\\D"))) +		if(sigmap(cemux->getPort(ID::Y)) != sigmap(count_reg->getPort(ID(D))))  			return 24;  		//Select of the mux is our clock enable  		extract.has_ce = true; -		extract.ce = sigmap(cemux->getPort("\\S")); +		extract.ce = sigmap(cemux->getPort(ID(S)));  	}  	else  		extract.has_ce = false;  	extract.count_reg = count_reg; -	if(count_reg->type == "$dff") +	if(count_reg->type == ID($dff))  		extract.has_reset = false; -	else if(count_reg->type == "$adff") +	else if(count_reg->type == ID($adff))  	{  		extract.has_reset = true;  		//Check polarity of reset - we may have to add an inverter later on! -		extract.rst_inverted = (count_reg->getParam("\\ARST_POLARITY").as_int() != 1); +		extract.rst_inverted = (count_reg->getParam(ID(ARST_POLARITY)).as_int() != 1);  		//Verify ARST_VALUE is zero or full scale -		int rst_value = count_reg->getParam("\\ARST_VALUE").as_int(); +		int rst_value = count_reg->getParam(ID(ARST_VALUE)).as_int();  		if(rst_value == 0)  			extract.rst_to_max = false;  		else if(rst_value == extract.count_value) @@ -251,7 +251,7 @@ int counter_tryextract(  			return 23;  		//Save the reset -		extract.rst = sigmap(count_reg->getPort("\\ARST")); +		extract.rst = sigmap(count_reg->getPort(ID(ARST)));  	}  	//TODO: support synchronous reset  	else @@ -260,12 +260,12 @@ int counter_tryextract(  	//Sanity check that we use the ALU output properly  	if(extract.has_ce)  	{ -		if(!is_full_bus(muxy, index, count_mux, "\\Y", cemux, "\\B")) +		if(!is_full_bus(muxy, index, count_mux, ID::Y, cemux, ID::B))  			return 16; -		if(!is_full_bus(cey, index, cemux, "\\Y", count_reg, "\\D")) +		if(!is_full_bus(cey, index, cemux, ID::Y, count_reg, ID(D)))  			return 16;  	} -	else if(!is_full_bus(muxy, index, count_mux, "\\Y", count_reg, "\\D")) +	else if(!is_full_bus(muxy, index, count_mux, ID::Y, count_reg, ID(D)))  		return 16;  	//TODO: Verify count_reg CLK_POLARITY is 1 @@ -273,7 +273,7 @@ int counter_tryextract(  	//Register output must have exactly two loads, the inverter and ALU  	//(unless we have a parallel output!)  	//If we have a clock enable, 3 is OK -	const RTLIL::SigSpec qport = count_reg->getPort("\\Q"); +	const RTLIL::SigSpec qport = count_reg->getPort(ID(Q));  	const RTLIL::SigSpec cnout = sigmap(qport);  	pool<Cell*> cnout_loads = get_other_cells(cnout, index, count_reg);  	unsigned int max_loads = 2; @@ -312,19 +312,19 @@ int counter_tryextract(  			}  		}  	} -	if(!is_full_bus(cnout, index, count_reg, "\\Q", underflow_inv, "\\A", true)) +	if(!is_full_bus(cnout, index, count_reg, ID(Q), underflow_inv, ID::A, true))  		return 18; -	if(!is_full_bus(cnout, index, count_reg, "\\Q", cell, "\\A", true)) +	if(!is_full_bus(cnout, index, count_reg, ID(Q), cell, ID::A, true))  		return 19;  	//Look up the clock from the register -	extract.clk = sigmap(count_reg->getPort("\\CLK")); +	extract.clk = sigmap(count_reg->getPort(ID(CLK)));  	//Register output net must have an INIT attribute equal to the count value  	extract.rwire = cnout.as_wire(); -	if(extract.rwire->attributes.find("\\init") == extract.rwire->attributes.end()) +	if(extract.rwire->attributes.find(ID(init)) == extract.rwire->attributes.end())  		return 20; -	int rinit = extract.rwire->attributes["\\init"].as_int(); +	int rinit = extract.rwire->attributes[ID(init)].as_int();  	if(rinit != extract.count_value)  		return 21; @@ -343,21 +343,21 @@ void counter_worker(  	SigMap& sigmap = index.sigmap;  	//Core of the counter must be an ALU -	if (cell->type != "$alu") +	if (cell->type != ID($alu))  		return;  	//A input is the count value. Check if it has COUNT_EXTRACT set.  	//If it's not a wire, don't even try -	auto port = sigmap(cell->getPort("\\A")); +	auto port = sigmap(cell->getPort(ID::A));  	if(!port.is_wire())  		return;  	RTLIL::Wire* a_wire = port.as_wire();  	bool force_extract = false;  	bool never_extract = false; -	string count_reg_src = a_wire->attributes["\\src"].decode_string().c_str(); -	if(a_wire->attributes.find("\\COUNT_EXTRACT") != a_wire->attributes.end()) +	string count_reg_src = a_wire->attributes[ID(src)].decode_string().c_str(); +	if(a_wire->attributes.find(ID(COUNT_EXTRACT)) != a_wire->attributes.end())  	{ -		pool<string> sa = a_wire->get_strpool_attribute("\\COUNT_EXTRACT"); +		pool<string> sa = a_wire->get_strpool_attribute(ID(COUNT_EXTRACT));  		string extract_value;  		if(sa.size() >= 1)  		{ @@ -434,66 +434,66 @@ void counter_worker(  	string countname = string("$COUNTx$") + log_id(extract.rwire->name.str());  	//Wipe all of the old connections to the ALU -	cell->unsetPort("\\A"); -	cell->unsetPort("\\B"); -	cell->unsetPort("\\BI"); -	cell->unsetPort("\\CI"); -	cell->unsetPort("\\CO"); -	cell->unsetPort("\\X"); -	cell->unsetPort("\\Y"); -	cell->unsetParam("\\A_SIGNED"); -	cell->unsetParam("\\A_WIDTH"); -	cell->unsetParam("\\B_SIGNED"); -	cell->unsetParam("\\B_WIDTH"); -	cell->unsetParam("\\Y_WIDTH"); +	cell->unsetPort(ID::A); +	cell->unsetPort(ID::B); +	cell->unsetPort(ID(BI)); +	cell->unsetPort(ID(CI)); +	cell->unsetPort(ID(CO)); +	cell->unsetPort(ID(X)); +	cell->unsetPort(ID::Y); +	cell->unsetParam(ID(A_SIGNED)); +	cell->unsetParam(ID(A_WIDTH)); +	cell->unsetParam(ID(B_SIGNED)); +	cell->unsetParam(ID(B_WIDTH)); +	cell->unsetParam(ID(Y_WIDTH));  	//Change the cell type -	cell->type = "$__COUNT_"; +	cell->type = ID($__COUNT_);  	//Hook up resets  	if(extract.has_reset)  	{  		//TODO: support other kinds of reset -		cell->setParam("\\RESET_MODE", RTLIL::Const("LEVEL")); +		cell->setParam(ID(RESET_MODE), RTLIL::Const("LEVEL"));  		//If the reset is active low, infer an inverter ($__COUNT_ cells always have active high reset)  		if(extract.rst_inverted)  		{  			auto realreset = cell->module->addWire(NEW_ID);  			cell->module->addNot(NEW_ID, extract.rst, RTLIL::SigSpec(realreset)); -			cell->setPort("\\RST", realreset); +			cell->setPort(ID(RST), realreset);  		}  		else -			cell->setPort("\\RST", extract.rst); +			cell->setPort(ID(RST), extract.rst);  	}  	else  	{ -		cell->setParam("\\RESET_MODE", RTLIL::Const("RISING")); -		cell->setPort("\\RST", RTLIL::SigSpec(false)); +		cell->setParam(ID(RESET_MODE), RTLIL::Const("RISING")); +		cell->setPort(ID(RST), RTLIL::SigSpec(false));  	}  	//Hook up other stuff -	//cell->setParam("\\CLKIN_DIVIDE", RTLIL::Const(1)); -	cell->setParam("\\COUNT_TO", RTLIL::Const(extract.count_value)); -	cell->setParam("\\WIDTH", RTLIL::Const(extract.width)); -	cell->setPort("\\CLK", extract.clk); -	cell->setPort("\\OUT", extract.outsig); +	//cell->setParam(ID(CLKIN_DIVIDE), RTLIL::Const(1)); +	cell->setParam(ID(COUNT_TO), RTLIL::Const(extract.count_value)); +	cell->setParam(ID(WIDTH), RTLIL::Const(extract.width)); +	cell->setPort(ID(CLK), extract.clk); +	cell->setPort(ID(OUT), extract.outsig);  	//Hook up clock enable  	if(extract.has_ce)  	{ -		cell->setParam("\\HAS_CE", RTLIL::Const(1)); -		cell->setPort("\\CE", extract.ce); +		cell->setParam(ID(HAS_CE), RTLIL::Const(1)); +		cell->setPort(ID(CE), extract.ce);  	}  	else -		cell->setParam("\\HAS_CE", RTLIL::Const(0)); +		cell->setParam(ID(HAS_CE), RTLIL::Const(0));  	//Hook up hard-wired ports (for now up/down are not supported), default to no parallel output -	cell->setParam("\\HAS_POUT", RTLIL::Const(0)); -	cell->setParam("\\RESET_TO_MAX", RTLIL::Const(0)); -	cell->setParam("\\DIRECTION", RTLIL::Const("DOWN")); -	cell->setPort("\\CE", RTLIL::Const(1)); -	cell->setPort("\\UP", RTLIL::Const(0)); +	cell->setParam(ID(HAS_POUT), RTLIL::Const(0)); +	cell->setParam(ID(RESET_TO_MAX), RTLIL::Const(0)); +	cell->setParam(ID(DIRECTION), RTLIL::Const("DOWN")); +	cell->setPort(ID(CE), RTLIL::Const(1)); +	cell->setPort(ID(UP), RTLIL::Const(0));  	//Hook up any parallel outputs  	for(auto load : extract.pouts) @@ -505,8 +505,8 @@ void counter_worker(  		//Connect it to our parallel output  		//(this is OK to do more than once b/c they all go to the same place) -		cell->setPort("\\POUT", sig); -		cell->setParam("\\HAS_POUT", RTLIL::Const(1)); +		cell->setPort(ID(POUT), sig); +		cell->setParam(ID(HAS_POUT), RTLIL::Const(1));  	}  	//Delete the cells we've replaced (let opt_clean handle deleting the now-redundant wires) @@ -546,7 +546,7 @@ void counter_worker(  		int newbits = ceil(log2(extract.count_value));  		if(extract.width != newbits)  		{ -			cell->setParam("\\WIDTH", RTLIL::Const(newbits)); +			cell->setParam(ID(WIDTH), RTLIL::Const(newbits));  			log("    Optimizing out %d unused high-order bits (new width is %d)\n",  				extract.width - newbits,  				newbits); diff --git a/passes/techmap/extract_fa.cc b/passes/techmap/extract_fa.cc index b541ceb6b..29700c37b 100644 --- a/passes/techmap/extract_fa.cc +++ b/passes/techmap/extract_fa.cc @@ -85,11 +85,11 @@ struct ExtractFaWorker  	{  		for (auto cell : module->selected_cells())  		{ -			if (cell->type.in( "$_BUF_", "$_NOT_", "$_AND_", "$_NAND_", "$_OR_", "$_NOR_", -					"$_XOR_", "$_XNOR_", "$_ANDNOT_", "$_ORNOT_", "$_MUX_", "$_NMUX_", -					"$_AOI3_", "$_OAI3_", "$_AOI4_", "$_OAI4_")) +			if (cell->type.in( ID($_BUF_), ID($_NOT_), ID($_AND_), ID($_NAND_), ID($_OR_), ID($_NOR_), +					ID($_XOR_), ID($_XNOR_), ID($_ANDNOT_), ID($_ORNOT_), ID($_MUX_), ID($_NMUX_), +					ID($_AOI3_), ID($_OAI3_), ID($_AOI4_), ID($_OAI4_)))  			{ -				SigBit y = sigmap(SigBit(cell->getPort("\\Y"))); +				SigBit y = sigmap(SigBit(cell->getPort(ID::Y)));  				log_assert(driver.count(y) == 0);  				driver[y] = cell;  			} @@ -262,10 +262,10 @@ struct ExtractFaWorker  			pool<SigBit> new_leaves = leaves;  			new_leaves.erase(bit); -			if (cell->hasPort("\\A")) new_leaves.insert(sigmap(SigBit(cell->getPort("\\A")))); -			if (cell->hasPort("\\B")) new_leaves.insert(sigmap(SigBit(cell->getPort("\\B")))); -			if (cell->hasPort("\\C")) new_leaves.insert(sigmap(SigBit(cell->getPort("\\C")))); -			if (cell->hasPort("\\D")) new_leaves.insert(sigmap(SigBit(cell->getPort("\\D")))); +			if (cell->hasPort(ID::A)) new_leaves.insert(sigmap(SigBit(cell->getPort(ID::A)))); +			if (cell->hasPort(ID::B)) new_leaves.insert(sigmap(SigBit(cell->getPort(ID::B)))); +			if (cell->hasPort(ID(C))) new_leaves.insert(sigmap(SigBit(cell->getPort(ID(C))))); +			if (cell->hasPort(ID(D))) new_leaves.insert(sigmap(SigBit(cell->getPort(ID(D)))));  			if (GetSize(new_leaves) > maxbreadth)  				continue; @@ -277,8 +277,8 @@ struct ExtractFaWorker  	void assign_new_driver(SigBit bit, SigBit new_driver)  	{  		Cell *cell = driver.at(bit); -		if (sigmap(cell->getPort("\\Y")) == bit) { -			cell->setPort("\\Y", module->addWire(NEW_ID)); +		if (sigmap(cell->getPort(ID::Y)) == bit) { +			cell->setPort(ID::Y, module->addWire(NEW_ID));  			module->connect(bit, new_driver);  		}  	} @@ -289,7 +289,7 @@ struct ExtractFaWorker  		for (auto it : driver)  		{ -			if (it.second->type.in("$_BUF_", "$_NOT_")) +			if (it.second->type.in(ID($_BUF_), ID($_NOT_)))  				continue;  			SigBit root = it.first; @@ -390,20 +390,20 @@ struct ExtractFaWorker  				}  				else  				{ -					Cell *cell = module->addCell(NEW_ID, "$fa"); -					cell->setParam("\\WIDTH", 1); +					Cell *cell = module->addCell(NEW_ID, ID($fa)); +					cell->setParam(ID(WIDTH), 1);  					log("      Created $fa cell %s.\n", log_id(cell)); -					cell->setPort("\\A", f3i.inv_a ? module->NotGate(NEW_ID, A) : A); -					cell->setPort("\\B", f3i.inv_b ? module->NotGate(NEW_ID, B) : B); -					cell->setPort("\\C", f3i.inv_c ? module->NotGate(NEW_ID, C) : C); +					cell->setPort(ID::A, f3i.inv_a ? module->NotGate(NEW_ID, A) : A); +					cell->setPort(ID::B, f3i.inv_b ? module->NotGate(NEW_ID, B) : B); +					cell->setPort(ID(C), f3i.inv_c ? module->NotGate(NEW_ID, C) : C);  					X = module->addWire(NEW_ID);  					Y = module->addWire(NEW_ID); -					cell->setPort("\\X", X); -					cell->setPort("\\Y", Y); +					cell->setPort(ID(X), X); +					cell->setPort(ID::Y, Y);  					facache[fakey] = make_tuple(X, Y, cell);  				} @@ -496,30 +496,30 @@ struct ExtractFaWorker  				}  				else  				{ -					Cell *cell = module->addCell(NEW_ID, "$fa"); -					cell->setParam("\\WIDTH", 1); +					Cell *cell = module->addCell(NEW_ID, ID($fa)); +					cell->setParam(ID(WIDTH), 1);  					log("      Created $fa cell %s.\n", log_id(cell)); -					cell->setPort("\\A", f2i.inv_a ? module->NotGate(NEW_ID, A) : A); -					cell->setPort("\\B", f2i.inv_b ? module->NotGate(NEW_ID, B) : B); -					cell->setPort("\\C", State::S0); +					cell->setPort(ID::A, f2i.inv_a ? module->NotGate(NEW_ID, A) : A); +					cell->setPort(ID::B, f2i.inv_b ? module->NotGate(NEW_ID, B) : B); +					cell->setPort(ID(C), State::S0);  					X = module->addWire(NEW_ID);  					Y = module->addWire(NEW_ID); -					cell->setPort("\\X", X); -					cell->setPort("\\Y", Y); +					cell->setPort(ID(X), X); +					cell->setPort(ID::Y, Y);  				}  				if (func2.at(key).count(xor2_func)) { -					SigBit YY = invert_xy ? module->NotGate(NEW_ID, Y) : Y; +					SigBit YY = invert_xy || (f2i.inv_a && !f2i.inv_b) || (!f2i.inv_a && f2i.inv_b) ? module->NotGate(NEW_ID, Y) : Y;  					for (auto bit : func2.at(key).at(xor2_func))  						assign_new_driver(bit, YY);  				}  				if (func2.at(key).count(xnor2_func)) { -					SigBit YY = invert_xy ? Y : module->NotGate(NEW_ID, Y); +					SigBit YY = invert_xy || (f2i.inv_a && !f2i.inv_b) || (!f2i.inv_a && f2i.inv_b) ? Y : module->NotGate(NEW_ID, Y);  					for (auto bit : func2.at(key).at(xnor2_func))  						assign_new_driver(bit, YY);  				} diff --git a/passes/techmap/extract_reduce.cc b/passes/techmap/extract_reduce.cc index a77bbc0b7..11cfddcd9 100644 --- a/passes/techmap/extract_reduce.cc +++ b/passes/techmap/extract_reduce.cc @@ -58,9 +58,9 @@ struct ExtractReducePass : public Pass  	inline bool IsRightType(Cell* cell, GateType gt)  	{ -		return (cell->type == "$_AND_" && gt == GateType::And) || -				(cell->type == "$_OR_" && gt == GateType::Or) || -				(cell->type == "$_XOR_" && gt == GateType::Xor); +		return (cell->type == ID($_AND_) && gt == GateType::And) || +				(cell->type == ID($_OR_) && gt == GateType::Or) || +				(cell->type == ID($_XOR_) && gt == GateType::Xor);  	}  	void execute(std::vector<std::string> args, RTLIL::Design *design) YS_OVERRIDE @@ -124,11 +124,11 @@ struct ExtractReducePass : public Pass  				GateType gt; -				if (cell->type == "$_AND_") +				if (cell->type == ID($_AND_))  					gt = GateType::And; -				else if (cell->type == "$_OR_") +				else if (cell->type == ID($_OR_))  					gt = GateType::Or; -				else if (cell->type == "$_XOR_") +				else if (cell->type == ID($_XOR_))  					gt = GateType::Xor;  				else  					continue; @@ -148,7 +148,7 @@ struct ExtractReducePass : public Pass  						head_cell = x; -						auto y = sigmap(x->getPort("\\Y")); +						auto y = sigmap(x->getPort(ID::Y));  						log_assert(y.size() == 1);  						// Should only continue if there is one fanout back into a cell (not to a port) @@ -166,7 +166,7 @@ struct ExtractReducePass : public Pass  				{  					//BFS, following all chains until they hit a cell of a different type  					//Pick the longest one -					auto y = sigmap(cell->getPort("\\Y")); +					auto y = sigmap(cell->getPort(ID::Y));  					pool<Cell*> current_loads = sig_to_sink[y];  					pool<Cell*> next_loads; @@ -233,7 +233,7 @@ struct ExtractReducePass : public Pass  						cur_supercell.insert(x); -						auto a = sigmap(x->getPort("\\A")); +						auto a = sigmap(x->getPort(ID::A));  						log_assert(a.size() == 1);  						// Must have only one sink unless we're going off chain @@ -249,7 +249,7 @@ struct ExtractReducePass : public Pass  							}  						} -						auto b = sigmap(x->getPort("\\B")); +						auto b = sigmap(x->getPort(ID::B));  						log_assert(b.size() == 1);  						// Must have only one sink @@ -279,26 +279,26 @@ struct ExtractReducePass : public Pass  						pool<SigBit> input_pool_intermed;  						for (auto x : cur_supercell)  						{ -							input_pool.insert(sigmap(x->getPort("\\A"))[0]); -							input_pool.insert(sigmap(x->getPort("\\B"))[0]); -							input_pool_intermed.insert(sigmap(x->getPort("\\Y"))[0]); +							input_pool.insert(sigmap(x->getPort(ID::A))[0]); +							input_pool.insert(sigmap(x->getPort(ID::B))[0]); +							input_pool_intermed.insert(sigmap(x->getPort(ID::Y))[0]);  						}  						SigSpec input;  						for (auto b : input_pool)  							if (input_pool_intermed.count(b) == 0)  								input.append_bit(b); -						SigBit output = sigmap(head_cell->getPort("\\Y")[0]); +						SigBit output = sigmap(head_cell->getPort(ID::Y)[0]);  						auto new_reduce_cell = module->addCell(NEW_ID, -							gt == GateType::And ? "$reduce_and" : -							gt == GateType::Or ? "$reduce_or" : -							gt == GateType::Xor ? "$reduce_xor" : ""); -						new_reduce_cell->setParam("\\A_SIGNED", 0); -						new_reduce_cell->setParam("\\A_WIDTH", input.size()); -						new_reduce_cell->setParam("\\Y_WIDTH", 1); -						new_reduce_cell->setPort("\\A", input); -						new_reduce_cell->setPort("\\Y", output); +							gt == GateType::And ? ID($reduce_and) : +							gt == GateType::Or ? ID($reduce_or) : +							gt == GateType::Xor ? ID($reduce_xor) : ""); +						new_reduce_cell->setParam(ID(A_SIGNED), 0); +						new_reduce_cell->setParam(ID(A_WIDTH), input.size()); +						new_reduce_cell->setParam(ID(Y_WIDTH), 1); +						new_reduce_cell->setPort(ID::A, input); +						new_reduce_cell->setPort(ID::Y, output);  						if(allow_off_chain)  							consumed_cells.insert(head_cell); diff --git a/passes/techmap/flowmap.cc b/passes/techmap/flowmap.cc index 96d0df5f8..5807178dd 100644 --- a/passes/techmap/flowmap.cc +++ b/passes/techmap/flowmap.cc @@ -671,8 +671,8 @@ struct FlowmapWorker  			labels[node] = -1;  		for (auto input : inputs)  		{ -			if (input.wire->attributes.count("\\$flowmap_level")) -				labels[input] = input.wire->attributes["\\$flowmap_level"].as_int(); +			if (input.wire->attributes.count(ID($flowmap_level))) +				labels[input] = input.wire->attributes[ID($flowmap_level)].as_int();  			else  				labels[input] = 0;  		} @@ -1412,7 +1412,7 @@ struct FlowmapWorker  			for (auto gate_node : lut_gates[node])  			{  				auto gate_origin = node_origins[gate_node]; -				lut->add_strpool_attribute("\\src", gate_origin.cell->get_strpool_attribute("\\src")); +				lut->add_strpool_attribute(ID(src), gate_origin.cell->get_strpool_attribute(ID(src)));  				packed_count++;  			}  			lut_count++; @@ -1586,7 +1586,7 @@ struct FlowmapPass : public Pass {  		}  		else  		{ -			cell_types = {"$_NOT_", "$_AND_", "$_OR_", "$_XOR_", "$_MUX_"}; +			cell_types = {ID($_NOT_), ID($_AND_), ID($_OR_), ID($_XOR_), ID($_MUX_)};  		}  		const char *algo_r = relax ? "-r" : ""; diff --git a/passes/techmap/iopadmap.cc b/passes/techmap/iopadmap.cc index efcc082d5..c868b9a87 100644 --- a/passes/techmap/iopadmap.cc +++ b/passes/techmap/iopadmap.cc @@ -64,6 +64,11 @@ struct IopadmapPass : public Pass {  		log("        of the tristate driver and the 2nd portname is the internal output\n");  		log("        buffering the external signal.\n");  		log("\n"); +		log("    -ignore <celltype> <portname>[:<portname>]*\n"); +		log("        Skips mapping inputs/outputs that are already connected to given\n"); +		log("        ports of the given cell.  Can be used multiple times.  This is in\n"); +		log("        addition to the cells specified as mapping targets.\n"); +		log("\n");  		log("    -widthparam <param_name>\n");  		log("        Use the specified parameter name to set the port width.\n");  		log("\n"); @@ -88,6 +93,7 @@ struct IopadmapPass : public Pass {  		std::string toutpad_celltype, toutpad_portname, toutpad_portname2, toutpad_portname3;  		std::string tinoutpad_celltype, tinoutpad_portname, tinoutpad_portname2, tinoutpad_portname3, tinoutpad_portname4;  		std::string widthparam, nameparam; +		pool<pair<IdString, IdString>> ignore;  		bool flag_bits = false;  		size_t argidx; @@ -127,6 +133,18 @@ struct IopadmapPass : public Pass {  				split_portname_pair(tinoutpad_portname3, tinoutpad_portname4);  				continue;  			} +			if (arg == "-ignore" && argidx+2 < args.size()) { +				std::string ignore_celltype = args[++argidx]; +				std::string ignore_portname = args[++argidx]; +				std::string ignore_portname2; +				while (!ignore_portname.empty()) { +					split_portname_pair(ignore_portname, ignore_portname2); +					ignore.insert(make_pair(RTLIL::escape_id(ignore_celltype), RTLIL::escape_id(ignore_portname))); + +					ignore_portname = ignore_portname2; +				} +				continue; +			}  			if (arg == "-widthparam" && argidx+1 < args.size()) {  				widthparam = args[++argidx];  				continue; @@ -143,6 +161,23 @@ struct IopadmapPass : public Pass {  		}  		extra_args(args, argidx, design); +		if (!inpad_portname2.empty()) +			ignore.insert(make_pair(RTLIL::escape_id(inpad_celltype), RTLIL::escape_id(inpad_portname2))); +		if (!outpad_portname2.empty()) +			ignore.insert(make_pair(RTLIL::escape_id(outpad_celltype), RTLIL::escape_id(outpad_portname2))); +		if (!inoutpad_portname2.empty()) +			ignore.insert(make_pair(RTLIL::escape_id(inoutpad_celltype), RTLIL::escape_id(inoutpad_portname2))); +		if (!toutpad_portname3.empty()) +			ignore.insert(make_pair(RTLIL::escape_id(toutpad_celltype), RTLIL::escape_id(toutpad_portname3))); +		if (!tinoutpad_portname4.empty()) +			ignore.insert(make_pair(RTLIL::escape_id(tinoutpad_celltype), RTLIL::escape_id(tinoutpad_portname4))); + +		for (auto module : design->modules()) +			if (module->get_blackbox_attribute()) +				for (auto wire : module->wires()) +					if (wire->get_bool_attribute("\\iopad_external_pin")) +						ignore.insert(make_pair(module->name, wire->name)); +  		for (auto module : design->selected_modules())  		{  			dict<IdString, pool<int>> skip_wires; @@ -150,28 +185,11 @@ struct IopadmapPass : public Pass {  			SigMap sigmap(module);  			for (auto cell : module->cells()) -			{ -				if (cell->type == RTLIL::escape_id(inpad_celltype) && cell->hasPort(RTLIL::escape_id(inpad_portname2))) -					for (auto bit : sigmap(cell->getPort(RTLIL::escape_id(inpad_portname2)))) -						skip_wire_bits.insert(bit); - -				if (cell->type == RTLIL::escape_id(outpad_celltype) && cell->hasPort(RTLIL::escape_id(outpad_portname2))) -					for (auto bit : sigmap(cell->getPort(RTLIL::escape_id(outpad_portname2)))) +			for (auto port : cell->connections()) +				if (ignore.count(make_pair(cell->type, port.first))) +					for (auto bit : sigmap(port.second))  						skip_wire_bits.insert(bit); -				if (cell->type == RTLIL::escape_id(inoutpad_celltype) && cell->hasPort(RTLIL::escape_id(inoutpad_portname2))) -					for (auto bit : sigmap(cell->getPort(RTLIL::escape_id(inoutpad_portname2)))) -						skip_wire_bits.insert(bit); - -				if (cell->type == RTLIL::escape_id(toutpad_celltype) && cell->hasPort(RTLIL::escape_id(toutpad_portname3))) -					for (auto bit : sigmap(cell->getPort(RTLIL::escape_id(toutpad_portname3)))) -						skip_wire_bits.insert(bit); - -				if (cell->type == RTLIL::escape_id(tinoutpad_celltype) && cell->hasPort(RTLIL::escape_id(tinoutpad_portname4))) -					for (auto bit : sigmap(cell->getPort(RTLIL::escape_id(tinoutpad_portname4)))) -						skip_wire_bits.insert(bit); -			} -  			if (!toutpad_celltype.empty() || !tinoutpad_celltype.empty())  			{  				dict<SigBit, pair<IdString, pool<IdString>>> tbuf_bits; @@ -179,8 +197,8 @@ struct IopadmapPass : public Pass {  				SigMap rewrites;  				for (auto cell : module->cells()) -					if (cell->type == "$_TBUF_") { -						SigBit bit = sigmap(cell->getPort("\\Y").as_bit()); +					if (cell->type == ID($_TBUF_)) { +						SigBit bit = sigmap(cell->getPort(ID::Y).as_bit());  						tbuf_bits[bit].first = cell->name;  					} @@ -212,8 +230,8 @@ struct IopadmapPass : public Pass {  						if (tbuf_cell == nullptr)  							continue; -						SigBit en_sig = tbuf_cell->getPort("\\E").as_bit(); -						SigBit data_sig = tbuf_cell->getPort("\\A").as_bit(); +						SigBit en_sig = tbuf_cell->getPort(ID(E)).as_bit(); +						SigBit data_sig = tbuf_cell->getPort(ID::A).as_bit();  						if (wire->port_input && !tinoutpad_celltype.empty())  						{ @@ -226,7 +244,7 @@ struct IopadmapPass : public Pass {  							cell->setPort(RTLIL::escape_id(tinoutpad_portname2), owire);  							cell->setPort(RTLIL::escape_id(tinoutpad_portname3), data_sig);  							cell->setPort(RTLIL::escape_id(tinoutpad_portname4), wire_bit); -							cell->attributes["\\keep"] = RTLIL::Const(1); +							cell->attributes[ID::keep] = RTLIL::Const(1);  							for (auto cn : tbuf_cache.second) {  								auto c = module->cell(cn); @@ -263,7 +281,7 @@ struct IopadmapPass : public Pass {  							cell->setPort(RTLIL::escape_id(toutpad_portname), en_sig);  							cell->setPort(RTLIL::escape_id(toutpad_portname2), data_sig);  							cell->setPort(RTLIL::escape_id(toutpad_portname3), wire_bit); -							cell->attributes["\\keep"] = RTLIL::Const(1); +							cell->attributes[ID::keep] = RTLIL::Const(1);  							for (auto cn : tbuf_cache.second) {  								auto c = module->cell(cn); @@ -390,7 +408,7 @@ struct IopadmapPass : public Pass {  							cell->parameters[RTLIL::escape_id(widthparam)] = RTLIL::Const(1);  						if (!nameparam.empty())  							cell->parameters[RTLIL::escape_id(nameparam)] = RTLIL::Const(stringf("%s[%d]", RTLIL::id2cstr(wire->name), i)); -						cell->attributes["\\keep"] = RTLIL::Const(1); +						cell->attributes[ID::keep] = RTLIL::Const(1);  					}  				}  				else @@ -403,7 +421,7 @@ struct IopadmapPass : public Pass {  						cell->parameters[RTLIL::escape_id(widthparam)] = RTLIL::Const(wire->width);  					if (!nameparam.empty())  						cell->parameters[RTLIL::escape_id(nameparam)] = RTLIL::Const(RTLIL::id2cstr(wire->name)); -					cell->attributes["\\keep"] = RTLIL::Const(1); +					cell->attributes[ID::keep] = RTLIL::Const(1);  				}  				wire->port_id = 0; diff --git a/passes/techmap/lut2mux.cc b/passes/techmap/lut2mux.cc index a4ed79550..c6618fc9d 100644 --- a/passes/techmap/lut2mux.cc +++ b/passes/techmap/lut2mux.cc @@ -25,9 +25,9 @@ PRIVATE_NAMESPACE_BEGIN  int lut2mux(Cell *cell)  { -	SigSpec sig_a = cell->getPort("\\A"); -	SigSpec sig_y = cell->getPort("\\Y"); -	Const lut = cell->getParam("\\LUT"); +	SigSpec sig_a = cell->getPort(ID::A); +	SigSpec sig_y = cell->getPort(ID::Y); +	Const lut = cell->getParam(ID(LUT));  	int count = 1;  	if (GetSize(sig_a) == 1) @@ -81,7 +81,7 @@ struct Lut2muxPass : public Pass {  		for (auto module : design->selected_modules())  		for (auto cell : module->selected_cells()) { -			if (cell->type == "$lut") { +			if (cell->type == ID($lut)) {  				IdString cell_name = cell->name;  				int count = lut2mux(cell);  				log("Converted %s.%s to %d MUX cells.\n", log_id(module), log_id(cell_name), count); diff --git a/passes/techmap/maccmap.cc b/passes/techmap/maccmap.cc index 59e58e4db..09f61927c 100644 --- a/passes/techmap/maccmap.cc +++ b/passes/techmap/maccmap.cc @@ -111,13 +111,13 @@ struct MaccmapWorker  			RTLIL::Wire *w1 = module->addWire(NEW_ID, width);  			RTLIL::Wire *w2 = module->addWire(NEW_ID, width); -			RTLIL::Cell *cell = module->addCell(NEW_ID, "$fa"); -			cell->setParam("\\WIDTH", width); -			cell->setPort("\\A", in1); -			cell->setPort("\\B", in2); -			cell->setPort("\\C", in3); -			cell->setPort("\\Y", w1); -			cell->setPort("\\X", w2); +			RTLIL::Cell *cell = module->addCell(NEW_ID, ID($fa)); +			cell->setParam(ID(WIDTH), width); +			cell->setPort(ID::A, in1); +			cell->setPort(ID::B, in2); +			cell->setPort(ID(C), in3); +			cell->setPort(ID::Y, w1); +			cell->setPort(ID(X), w2);  			out1 = {out_zeros_msb, w1, out_zeros_lsb};  			out2 = {out_zeros_msb, w2, out_zeros_lsb}; @@ -237,23 +237,23 @@ struct MaccmapWorker  		} -		RTLIL::Cell *c = module->addCell(NEW_ID, "$alu"); -		c->setPort("\\A", summands.front()); -		c->setPort("\\B", summands.back()); -		c->setPort("\\CI", State::S0); -		c->setPort("\\BI", State::S0); -		c->setPort("\\Y", module->addWire(NEW_ID, width)); -		c->setPort("\\X", module->addWire(NEW_ID, width)); -		c->setPort("\\CO", module->addWire(NEW_ID, width)); +		RTLIL::Cell *c = module->addCell(NEW_ID, ID($alu)); +		c->setPort(ID::A, summands.front()); +		c->setPort(ID::B, summands.back()); +		c->setPort(ID(CI), State::S0); +		c->setPort(ID(BI), State::S0); +		c->setPort(ID::Y, module->addWire(NEW_ID, width)); +		c->setPort(ID(X), module->addWire(NEW_ID, width)); +		c->setPort(ID(CO), module->addWire(NEW_ID, width));  		c->fixup_parameters();  		if (!tree_sum_bits.empty()) { -			c->setPort("\\CI", tree_sum_bits.back()); +			c->setPort(ID(CI), tree_sum_bits.back());  			tree_sum_bits.pop_back();  		}  		log_assert(tree_sum_bits.empty()); -		return c->getPort("\\Y"); +		return c->getPort(ID::Y);  	}  }; @@ -264,17 +264,17 @@ extern void maccmap(RTLIL::Module *module, RTLIL::Cell *cell, bool unmap = false  void maccmap(RTLIL::Module *module, RTLIL::Cell *cell, bool unmap)  { -	int width = GetSize(cell->getPort("\\Y")); +	int width = GetSize(cell->getPort(ID::Y));  	Macc macc;  	macc.from_cell(cell);  	RTLIL::SigSpec all_input_bits; -	all_input_bits.append(cell->getPort("\\A")); -	all_input_bits.append(cell->getPort("\\B")); +	all_input_bits.append(cell->getPort(ID::A)); +	all_input_bits.append(cell->getPort(ID::B));  	if (all_input_bits.to_sigbit_set().count(RTLIL::Sx)) { -		module->connect(cell->getPort("\\Y"), RTLIL::SigSpec(RTLIL::Sx, width)); +		module->connect(cell->getPort(ID::Y), RTLIL::SigSpec(RTLIL::Sx, width));  		return;  	} @@ -339,9 +339,9 @@ void maccmap(RTLIL::Module *module, RTLIL::Cell *cell, bool unmap)  		}  		if (summands.front().second) -			module->addNeg(NEW_ID, summands.front().first, cell->getPort("\\Y")); +			module->addNeg(NEW_ID, summands.front().first, cell->getPort(ID::Y));  		else -			module->connect(cell->getPort("\\Y"), summands.front().first); +			module->connect(cell->getPort(ID::Y), summands.front().first);  	}  	else  	{ @@ -356,7 +356,7 @@ void maccmap(RTLIL::Module *module, RTLIL::Cell *cell, bool unmap)  		for (auto &bit : macc.bit_ports)  			worker.add(bit, 0); -		module->connect(cell->getPort("\\Y"), worker.synth()); +		module->connect(cell->getPort(ID::Y), worker.synth());  	}  } @@ -393,7 +393,7 @@ struct MaccmapPass : public Pass {  		for (auto mod : design->selected_modules())  		for (auto cell : mod->selected_cells()) -			if (cell->type == "$macc") { +			if (cell->type == ID($macc)) {  				log("Mapping %s.%s (%s).\n", log_id(mod), log_id(cell), log_id(cell->type));  				maccmap(mod, cell, unmap_mode);  				mod->remove(cell); diff --git a/passes/techmap/muxcover.cc b/passes/techmap/muxcover.cc index 81546249c..5541b6122 100644 --- a/passes/techmap/muxcover.cc +++ b/passes/techmap/muxcover.cc @@ -116,13 +116,13 @@ struct MuxcoverWorker  				if (!cell->input(conn.first))  					continue;  				for (auto bit : sigmap(conn.second)) { -					if (used_once.count(bit) || cell->type != "$_MUX_" || conn.first == "\\S") +					if (used_once.count(bit) || cell->type != ID($_MUX_) || conn.first == ID(S))  						roots.insert(bit);  					used_once.insert(bit);  				}  			} -			if (cell->type == "$_MUX_") -				sig_to_mux[sigmap(cell->getPort("\\Y"))] = cell; +			if (cell->type == ID($_MUX_)) +				sig_to_mux[sigmap(cell->getPort(ID::Y))] = cell;  		}  		log("  Treeifying %d MUXes:\n", GetSize(sig_to_mux)); @@ -141,8 +141,8 @@ struct MuxcoverWorker  				if (sig_to_mux.count(bit) && (bit == rootsig || !roots.count(bit))) {  					Cell *c = sig_to_mux.at(bit);  					tree.muxes[bit] = c; -					wavefront.insert(sigmap(c->getPort("\\A"))); -					wavefront.insert(sigmap(c->getPort("\\B"))); +					wavefront.insert(sigmap(c->getPort(ID::A))); +					wavefront.insert(sigmap(c->getPort(ID::B)));  				}  			} @@ -516,69 +516,69 @@ struct MuxcoverWorker  		if (GetSize(mux.inputs) == 2) {  			count_muxes_by_type[0]++; -			Cell *cell = module->addCell(NEW_ID, "$_MUX_"); -			cell->setPort("\\A", mux.inputs[0]); -			cell->setPort("\\B", mux.inputs[1]); -			cell->setPort("\\S", mux.selects[0]); -			cell->setPort("\\Y", bit); +			Cell *cell = module->addCell(NEW_ID, ID($_MUX_)); +			cell->setPort(ID::A, mux.inputs[0]); +			cell->setPort(ID::B, mux.inputs[1]); +			cell->setPort(ID(S), mux.selects[0]); +			cell->setPort(ID::Y, bit);  			return;  		}  		if (GetSize(mux.inputs) == 4) {  			count_muxes_by_type[1]++; -			Cell *cell = module->addCell(NEW_ID, "$_MUX4_"); -			cell->setPort("\\A", mux.inputs[0]); -			cell->setPort("\\B", mux.inputs[1]); -			cell->setPort("\\C", mux.inputs[2]); -			cell->setPort("\\D", mux.inputs[3]); -			cell->setPort("\\S", mux.selects[0]); -			cell->setPort("\\T", mux.selects[1]); -			cell->setPort("\\Y", bit); +			Cell *cell = module->addCell(NEW_ID, ID($_MUX4_)); +			cell->setPort(ID::A, mux.inputs[0]); +			cell->setPort(ID::B, mux.inputs[1]); +			cell->setPort(ID(C), mux.inputs[2]); +			cell->setPort(ID(D), mux.inputs[3]); +			cell->setPort(ID(S), mux.selects[0]); +			cell->setPort(ID(T), mux.selects[1]); +			cell->setPort(ID::Y, bit);  			return;  		}  		if (GetSize(mux.inputs) == 8) {  			count_muxes_by_type[2]++; -			Cell *cell = module->addCell(NEW_ID, "$_MUX8_"); -			cell->setPort("\\A", mux.inputs[0]); -			cell->setPort("\\B", mux.inputs[1]); -			cell->setPort("\\C", mux.inputs[2]); -			cell->setPort("\\D", mux.inputs[3]); -			cell->setPort("\\E", mux.inputs[4]); -			cell->setPort("\\F", mux.inputs[5]); -			cell->setPort("\\G", mux.inputs[6]); -			cell->setPort("\\H", mux.inputs[7]); -			cell->setPort("\\S", mux.selects[0]); -			cell->setPort("\\T", mux.selects[1]); -			cell->setPort("\\U", mux.selects[2]); -			cell->setPort("\\Y", bit); +			Cell *cell = module->addCell(NEW_ID, ID($_MUX8_)); +			cell->setPort(ID::A, mux.inputs[0]); +			cell->setPort(ID::B, mux.inputs[1]); +			cell->setPort(ID(C), mux.inputs[2]); +			cell->setPort(ID(D), mux.inputs[3]); +			cell->setPort(ID(E), mux.inputs[4]); +			cell->setPort(ID(F), mux.inputs[5]); +			cell->setPort(ID(G), mux.inputs[6]); +			cell->setPort(ID(H), mux.inputs[7]); +			cell->setPort(ID(S), mux.selects[0]); +			cell->setPort(ID(T), mux.selects[1]); +			cell->setPort(ID(U), mux.selects[2]); +			cell->setPort(ID::Y, bit);  			return;  		}  		if (GetSize(mux.inputs) == 16) {  			count_muxes_by_type[3]++; -			Cell *cell = module->addCell(NEW_ID, "$_MUX16_"); -			cell->setPort("\\A", mux.inputs[0]); -			cell->setPort("\\B", mux.inputs[1]); -			cell->setPort("\\C", mux.inputs[2]); -			cell->setPort("\\D", mux.inputs[3]); -			cell->setPort("\\E", mux.inputs[4]); -			cell->setPort("\\F", mux.inputs[5]); -			cell->setPort("\\G", mux.inputs[6]); -			cell->setPort("\\H", mux.inputs[7]); -			cell->setPort("\\I", mux.inputs[8]); -			cell->setPort("\\J", mux.inputs[9]); -			cell->setPort("\\K", mux.inputs[10]); -			cell->setPort("\\L", mux.inputs[11]); -			cell->setPort("\\M", mux.inputs[12]); -			cell->setPort("\\N", mux.inputs[13]); -			cell->setPort("\\O", mux.inputs[14]); -			cell->setPort("\\P", mux.inputs[15]); -			cell->setPort("\\S", mux.selects[0]); -			cell->setPort("\\T", mux.selects[1]); -			cell->setPort("\\U", mux.selects[2]); -			cell->setPort("\\V", mux.selects[3]); -			cell->setPort("\\Y", bit); +			Cell *cell = module->addCell(NEW_ID, ID($_MUX16_)); +			cell->setPort(ID::A, mux.inputs[0]); +			cell->setPort(ID::B, mux.inputs[1]); +			cell->setPort(ID(C), mux.inputs[2]); +			cell->setPort(ID(D), mux.inputs[3]); +			cell->setPort(ID(E), mux.inputs[4]); +			cell->setPort(ID(F), mux.inputs[5]); +			cell->setPort(ID(G), mux.inputs[6]); +			cell->setPort(ID(H), mux.inputs[7]); +			cell->setPort(ID(I), mux.inputs[8]); +			cell->setPort(ID(J), mux.inputs[9]); +			cell->setPort(ID(K), mux.inputs[10]); +			cell->setPort(ID(L), mux.inputs[11]); +			cell->setPort(ID(M), mux.inputs[12]); +			cell->setPort(ID(N), mux.inputs[13]); +			cell->setPort(ID(O), mux.inputs[14]); +			cell->setPort(ID(P), mux.inputs[15]); +			cell->setPort(ID(S), mux.selects[0]); +			cell->setPort(ID(T), mux.selects[1]); +			cell->setPort(ID(U), mux.selects[2]); +			cell->setPort(ID(V), mux.selects[3]); +			cell->setPort(ID::Y, bit);  			return;  		} diff --git a/passes/techmap/nlutmap.cc b/passes/techmap/nlutmap.cc index cc765d89c..798d82248 100644 --- a/passes/techmap/nlutmap.cc +++ b/passes/techmap/nlutmap.cc @@ -82,10 +82,10 @@ struct NlutmapWorker  			for (auto cell : module->cells())  			{ -				if (cell->type != "$lut" || mapped_cells.count(cell)) +				if (cell->type != ID($lut) || mapped_cells.count(cell))  					continue; -				if (GetSize(cell->getPort("\\A")) == lut_size || lut_size == 2) +				if (GetSize(cell->getPort(ID::A)) == lut_size || lut_size == 2)  					candidate_ratings[cell] = 0;  				for (auto &conn : cell->connections()) @@ -119,7 +119,7 @@ struct NlutmapWorker  		if (config.assert_mode) {  			for (auto cell : module->cells()) -				if (cell->type == "$lut" && !mapped_cells.count(cell)) +				if (cell->type == ID($lut) && !mapped_cells.count(cell))  					log_error("Insufficient number of LUTs to map all logic cells!\n");  		} diff --git a/passes/techmap/pmuxtree.cc b/passes/techmap/pmuxtree.cc index 6a923f481..31ab13cec 100644 --- a/passes/techmap/pmuxtree.cc +++ b/passes/techmap/pmuxtree.cc @@ -89,21 +89,21 @@ struct PmuxtreePass : public Pass {  		for (auto module : design->selected_modules())  		for (auto cell : module->selected_cells())  		{ -			if (cell->type != "$pmux") +			if (cell->type != ID($pmux))  				continue; -			SigSpec sig_data = cell->getPort("\\B"); -			SigSpec sig_sel = cell->getPort("\\S"); +			SigSpec sig_data = cell->getPort(ID::B); +			SigSpec sig_sel = cell->getPort(ID(S)); -			if (!cell->getPort("\\A").is_fully_undef()) { -				sig_data.append(cell->getPort("\\A")); +			if (!cell->getPort(ID::A).is_fully_undef()) { +				sig_data.append(cell->getPort(ID::A));  				SigSpec sig_sel_or = module->ReduceOr(NEW_ID, sig_sel);  				sig_sel.append(module->Not(NEW_ID, sig_sel_or));  			}  			SigSpec result, result_or;  			result = recursive_mux_generator(module, sig_data, sig_sel, result_or); -			module->connect(cell->getPort("\\Y"), result); +			module->connect(cell->getPort(ID::Y), result);  			module->remove(cell);  		}  	} diff --git a/passes/techmap/shregmap.cc b/passes/techmap/shregmap.cc index 004ab1eb9..02cc27ae2 100644 --- a/passes/techmap/shregmap.cc +++ b/passes/techmap/shregmap.cc @@ -73,22 +73,22 @@ struct ShregmapTechGreenpak4 : ShregmapTech  	bool fixup(Cell *cell, dict<int, SigBit> &taps)  	{ -		auto D = cell->getPort("\\D"); -		auto C = cell->getPort("\\C"); +		auto D = cell->getPort(ID(D)); +		auto C = cell->getPort(ID(C)); -		auto newcell = cell->module->addCell(NEW_ID, "\\GP_SHREG"); -		newcell->setPort("\\nRST", State::S1); -		newcell->setPort("\\CLK", C); -		newcell->setPort("\\IN", D); +		auto newcell = cell->module->addCell(NEW_ID, ID(GP_SHREG)); +		newcell->setPort(ID(nRST), State::S1); +		newcell->setPort(ID(CLK), C); +		newcell->setPort(ID(IN), D);  		int i = 0;  		for (auto tap : taps) { -			newcell->setPort(i ? "\\OUTB" : "\\OUTA", tap.second); -			newcell->setParam(i ? "\\OUTB_TAP" : "\\OUTA_TAP", tap.first + 1); +			newcell->setPort(i ? ID(OUTB) : ID(OUTA), tap.second); +			newcell->setParam(i ? ID(OUTB_TAP) : ID(OUTA_TAP), tap.first + 1);  			i++;  		} -		cell->setParam("\\OUTA_INVERT", 0); +		cell->setParam(ID(OUTA_INVERT), 0);  		return false;  	}  }; @@ -104,19 +104,19 @@ struct ShregmapTechXilinx7 : ShregmapTech  	{  		for (const auto &i : module->cells_) {  			auto cell = i.second; -			if (cell->type == "$shiftx") { -				if (cell->getParam("\\Y_WIDTH") != 1) continue; +			if (cell->type == ID($shiftx)) { +				if (cell->getParam(ID(Y_WIDTH)) != 1) continue;  				int j = 0; -				for (auto bit : sigmap(cell->getPort("\\A"))) +				for (auto bit : sigmap(cell->getPort(ID::A)))  					sigbit_to_shiftx_offset[bit] = std::make_tuple(cell, j++, 0); -				log_assert(j == cell->getParam("\\A_WIDTH").as_int()); +				log_assert(j == cell->getParam(ID(A_WIDTH)).as_int());  			} -			else if (cell->type == "$mux") { +			else if (cell->type == ID($mux)) {  				int j = 0; -				for (auto bit : sigmap(cell->getPort("\\A"))) +				for (auto bit : sigmap(cell->getPort(ID::A)))  					sigbit_to_shiftx_offset[bit] = std::make_tuple(cell, 0, j++);  				j = 0; -				for (auto bit : sigmap(cell->getPort("\\B"))) +				for (auto bit : sigmap(cell->getPort(ID::B)))  					sigbit_to_shiftx_offset[bit] = std::make_tuple(cell, 1, j++);  			}  		} @@ -128,9 +128,9 @@ struct ShregmapTechXilinx7 : ShregmapTech  		if (it == sigbit_to_shiftx_offset.end())  			return;  		if (cell) { -			if (cell->type == "$shiftx" && port == "\\A") +			if (cell->type == ID($shiftx) && port == ID::A)  				return; -			if (cell->type == "$mux" && (port == "\\A" || port == "\\B")) +			if (cell->type == ID($mux) && port.in(ID::A, ID::B))  				return;  		}  		sigbit_to_shiftx_offset.erase(it); @@ -177,21 +177,21 @@ struct ShregmapTechXilinx7 : ShregmapTech  		log_assert(shiftx);  		// Only map if $shiftx exclusively covers the shift register -		if (shiftx->type == "$shiftx") { -			if (GetSize(taps) > shiftx->getParam("\\A_WIDTH").as_int()) +		if (shiftx->type == ID($shiftx)) { +			if (GetSize(taps) > shiftx->getParam(ID(A_WIDTH)).as_int())  				return false;  			// Due to padding the most significant bits of A may be 1'bx,  			//   and if so, discount them -			if (GetSize(taps) < shiftx->getParam("\\A_WIDTH").as_int()) { -				const SigSpec A = shiftx->getPort("\\A"); -				const int A_width = shiftx->getParam("\\A_WIDTH").as_int(); +			if (GetSize(taps) < shiftx->getParam(ID(A_WIDTH)).as_int()) { +				const SigSpec A = shiftx->getPort(ID::A); +				const int A_width = shiftx->getParam(ID(A_WIDTH)).as_int();  				for (int i = GetSize(taps); i < A_width; ++i)  					if (A[i] != RTLIL::Sx) return false;  			} -			else if (GetSize(taps) != shiftx->getParam("\\A_WIDTH").as_int()) +			else if (GetSize(taps) != shiftx->getParam(ID(A_WIDTH)).as_int())  				return false;  		} -		else if (shiftx->type == "$mux") { +		else if (shiftx->type == ID($mux)) {  			if (GetSize(taps) != 2)  				return false;  		} @@ -208,34 +208,34 @@ struct ShregmapTechXilinx7 : ShregmapTech  		auto it = sigbit_to_shiftx_offset.find(bit);  		log_assert(it != sigbit_to_shiftx_offset.end()); -		auto newcell = cell->module->addCell(NEW_ID, "$__XILINX_SHREG_"); +		auto newcell = cell->module->addCell(NEW_ID, ID($__XILINX_SHREG_));  		newcell->set_src_attribute(cell->get_src_attribute()); -		newcell->setParam("\\DEPTH", cell->getParam("\\DEPTH")); -		newcell->setParam("\\INIT", cell->getParam("\\INIT")); -		newcell->setParam("\\CLKPOL", cell->getParam("\\CLKPOL")); -		newcell->setParam("\\ENPOL", cell->getParam("\\ENPOL")); +		newcell->setParam(ID(DEPTH), cell->getParam(ID(DEPTH))); +		newcell->setParam(ID(INIT), cell->getParam(ID(INIT))); +		newcell->setParam(ID(CLKPOL), cell->getParam(ID(CLKPOL))); +		newcell->setParam(ID(ENPOL), cell->getParam(ID(ENPOL))); -		newcell->setPort("\\C", cell->getPort("\\C")); -		newcell->setPort("\\D", cell->getPort("\\D")); -		if (cell->hasPort("\\E")) -			newcell->setPort("\\E", cell->getPort("\\E")); +		newcell->setPort(ID(C), cell->getPort(ID(C))); +		newcell->setPort(ID(D), cell->getPort(ID(D))); +		if (cell->hasPort(ID(E))) +			newcell->setPort(ID(E), cell->getPort(ID(E)));  		Cell* shiftx = std::get<0>(it->second);  		RTLIL::SigSpec l_wire, q_wire; -		if (shiftx->type == "$shiftx") { -			l_wire = shiftx->getPort("\\B"); -			q_wire = shiftx->getPort("\\Y"); -			shiftx->setPort("\\Y", cell->module->addWire(NEW_ID)); +		if (shiftx->type == ID($shiftx)) { +			l_wire = shiftx->getPort(ID::B); +			q_wire = shiftx->getPort(ID::Y); +			shiftx->setPort(ID::Y, cell->module->addWire(NEW_ID));  		} -		else if (shiftx->type == "$mux") { -			l_wire = shiftx->getPort("\\S"); -			q_wire = shiftx->getPort("\\Y"); -			shiftx->setPort("\\Y", cell->module->addWire(NEW_ID)); +		else if (shiftx->type == ID($mux)) { +			l_wire = shiftx->getPort(ID(S)); +			q_wire = shiftx->getPort(ID::Y); +			shiftx->setPort(ID::Y, cell->module->addWire(NEW_ID));  		}  		else log_abort(); -		newcell->setPort("\\Q", q_wire); -		newcell->setPort("\\L", l_wire); +		newcell->setPort(ID(Q), q_wire); +		newcell->setPort(ID(L), l_wire);  		return false;  	} @@ -263,16 +263,16 @@ struct ShregmapWorker  	{  		for (auto wire : module->wires())  		{ -			if (wire->port_output || wire->get_bool_attribute("\\keep")) { +			if (wire->port_output || wire->get_bool_attribute(ID::keep)) {  				for (auto bit : sigmap(wire)) {  					sigbit_with_non_chain_users.insert(bit);  					if (opts.tech) opts.tech->non_chain_user(bit, nullptr, {});  				}  			} -			if (wire->attributes.count("\\init")) { +			if (wire->attributes.count(ID(init))) {  				SigSpec initsig = sigmap(wire); -				Const initval = wire->attributes.at("\\init"); +				Const initval = wire->attributes.at(ID(init));  				for (int i = 0; i < GetSize(initsig) && i < GetSize(initval); i++)  					if (initval[i] == State::S0 && !opts.zinit)  						sigbit_init[initsig[i]] = false; @@ -283,7 +283,7 @@ struct ShregmapWorker  		for (auto cell : module->cells())  		{ -			if (opts.ffcells.count(cell->type) && !cell->get_bool_attribute("\\keep")) +			if (opts.ffcells.count(cell->type) && !cell->get_bool_attribute(ID::keep))  			{  				IdString d_port = opts.ffcells.at(cell->type).first;  				IdString q_port = opts.ffcells.at(cell->type).second; @@ -346,7 +346,7 @@ struct ShregmapWorker  				IdString q_port = opts.ffcells.at(c1->type).second;  				auto c1_conn = c1->connections(); -				auto c2_conn = c1->connections(); +				auto c2_conn = c2->connections();  				c1_conn.erase(d_port);  				c1_conn.erase(q_port); @@ -474,7 +474,7 @@ struct ShregmapWorker  						initval.push_back(State::S0);  					remove_init.insert(bit);  				} -				first_cell->setParam("\\INIT", initval); +				first_cell->setParam(ID(INIT), initval);  			}  			if (opts.zinit) @@ -488,22 +488,22 @@ struct ShregmapWorker  				int param_clkpol = -1;  				int param_enpol = 2; -				if (first_cell->type == "$_DFF_N_") param_clkpol = 0; -				if (first_cell->type == "$_DFF_P_") param_clkpol = 1; +				if (first_cell->type == ID($_DFF_N_)) param_clkpol = 0; +				if (first_cell->type == ID($_DFF_P_)) param_clkpol = 1; -				if (first_cell->type == "$_DFFE_NN_") param_clkpol = 0, param_enpol = 0; -				if (first_cell->type == "$_DFFE_NP_") param_clkpol = 0, param_enpol = 1; -				if (first_cell->type == "$_DFFE_PN_") param_clkpol = 1, param_enpol = 0; -				if (first_cell->type == "$_DFFE_PP_") param_clkpol = 1, param_enpol = 1; +				if (first_cell->type == ID($_DFFE_NN_)) param_clkpol = 0, param_enpol = 0; +				if (first_cell->type == ID($_DFFE_NP_)) param_clkpol = 0, param_enpol = 1; +				if (first_cell->type == ID($_DFFE_PN_)) param_clkpol = 1, param_enpol = 0; +				if (first_cell->type == ID($_DFFE_PP_)) param_clkpol = 1, param_enpol = 1;  				log_assert(param_clkpol >= 0); -				first_cell->setParam("\\CLKPOL", param_clkpol); -				if (opts.ffe) first_cell->setParam("\\ENPOL", param_enpol); +				first_cell->setParam(ID(CLKPOL), param_clkpol); +				if (opts.ffe) first_cell->setParam(ID(ENPOL), param_enpol);  			}  			first_cell->type = shreg_cell_type_str;  			first_cell->setPort(q_port, last_cell->getPort(q_port)); -			first_cell->setParam("\\DEPTH", depth); +			first_cell->setParam(ID(DEPTH), depth);  			if (opts.tech != nullptr && !opts.tech->fixup(first_cell, taps_dict))  				remove_cells.insert(first_cell); @@ -521,18 +521,18 @@ struct ShregmapWorker  		for (auto wire : module->wires())  		{ -			if (wire->attributes.count("\\init") == 0) +			if (wire->attributes.count(ID(init)) == 0)  				continue;  			SigSpec initsig = sigmap(wire); -			Const &initval = wire->attributes.at("\\init"); +			Const &initval = wire->attributes.at(ID(init));  			for (int i = 0; i < GetSize(initsig) && i < GetSize(initval); i++)  				if (remove_init.count(initsig[i]))  					initval[i] = State::Sx;  			if (SigSpec(initval).is_fully_undef()) -				wire->attributes.erase("\\init"); +				wire->attributes.erase(ID(init));  		}  		remove_cells.clear(); @@ -717,19 +717,19 @@ struct ShregmapPass : public Pass {  			bool en_neg = enpol == "neg" || enpol == "any" || enpol == "any_or_none";  			if (clk_pos && en_none) -				opts.ffcells["$_DFF_P_"] = make_pair(IdString("\\D"), IdString("\\Q")); +				opts.ffcells[ID($_DFF_P_)] = make_pair(IdString(ID(D)), IdString(ID(Q)));  			if (clk_neg && en_none) -				opts.ffcells["$_DFF_N_"] = make_pair(IdString("\\D"), IdString("\\Q")); +				opts.ffcells[ID($_DFF_N_)] = make_pair(IdString(ID(D)), IdString(ID(Q)));  			if (clk_pos && en_pos) -				opts.ffcells["$_DFFE_PP_"] = make_pair(IdString("\\D"), IdString("\\Q")); +				opts.ffcells[ID($_DFFE_PP_)] = make_pair(IdString(ID(D)), IdString(ID(Q)));  			if (clk_pos && en_neg) -				opts.ffcells["$_DFFE_PN_"] = make_pair(IdString("\\D"), IdString("\\Q")); +				opts.ffcells[ID($_DFFE_PN_)] = make_pair(IdString(ID(D)), IdString(ID(Q)));  			if (clk_neg && en_pos) -				opts.ffcells["$_DFFE_NP_"] = make_pair(IdString("\\D"), IdString("\\Q")); +				opts.ffcells[ID($_DFFE_NP_)] = make_pair(IdString(ID(D)), IdString(ID(Q)));  			if (clk_neg && en_neg) -				opts.ffcells["$_DFFE_NN_"] = make_pair(IdString("\\D"), IdString("\\Q")); +				opts.ffcells[ID($_DFFE_NN_)] = make_pair(IdString(ID(D)), IdString(ID(Q)));  			if (en_pos || en_neg)  				opts.ffe = true; diff --git a/passes/techmap/simplemap.cc b/passes/techmap/simplemap.cc index 2eaba1b09..91574f3c6 100644 --- a/passes/techmap/simplemap.cc +++ b/passes/techmap/simplemap.cc @@ -28,82 +28,82 @@ YOSYS_NAMESPACE_BEGIN  void simplemap_not(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y); -	sig_a.extend_u0(GetSize(sig_y), cell->parameters.at("\\A_SIGNED").as_bool()); +	sig_a.extend_u0(GetSize(sig_y), cell->parameters.at(ID(A_SIGNED)).as_bool());  	for (int i = 0; i < GetSize(sig_y); i++) { -		RTLIL::Cell *gate = module->addCell(NEW_ID, "$_NOT_"); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\A", sig_a[i]); -		gate->setPort("\\Y", sig_y[i]); +		RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_NOT_)); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID::A, sig_a[i]); +		gate->setPort(ID::Y, sig_y[i]);  	}  }  void simplemap_pos(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y); -	sig_a.extend_u0(GetSize(sig_y), cell->parameters.at("\\A_SIGNED").as_bool()); +	sig_a.extend_u0(GetSize(sig_y), cell->parameters.at(ID(A_SIGNED)).as_bool());  	module->connect(RTLIL::SigSig(sig_y, sig_a));  }  void simplemap_bitop(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_b = cell->getPort("\\B"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_b = cell->getPort(ID::B); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y); -	sig_a.extend_u0(GetSize(sig_y), cell->parameters.at("\\A_SIGNED").as_bool()); -	sig_b.extend_u0(GetSize(sig_y), cell->parameters.at("\\B_SIGNED").as_bool()); +	sig_a.extend_u0(GetSize(sig_y), cell->parameters.at(ID(A_SIGNED)).as_bool()); +	sig_b.extend_u0(GetSize(sig_y), cell->parameters.at(ID(B_SIGNED)).as_bool()); -	if (cell->type == "$xnor") +	if (cell->type == ID($xnor))  	{  		RTLIL::SigSpec sig_t = module->addWire(NEW_ID, GetSize(sig_y));  		for (int i = 0; i < GetSize(sig_y); i++) { -			RTLIL::Cell *gate = module->addCell(NEW_ID, "$_NOT_"); -			gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -			gate->setPort("\\A", sig_t[i]); -			gate->setPort("\\Y", sig_y[i]); +			RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_NOT_)); +			gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +			gate->setPort(ID::A, sig_t[i]); +			gate->setPort(ID::Y, sig_y[i]);  		}  		sig_y = sig_t;  	} -	std::string gate_type; -	if (cell->type == "$and")  gate_type = "$_AND_"; -	if (cell->type == "$or")   gate_type = "$_OR_"; -	if (cell->type == "$xor")  gate_type = "$_XOR_"; -	if (cell->type == "$xnor") gate_type = "$_XOR_"; +	IdString gate_type; +	if (cell->type == ID($and))  gate_type = ID($_AND_); +	if (cell->type == ID($or))   gate_type = ID($_OR_); +	if (cell->type == ID($xor))  gate_type = ID($_XOR_); +	if (cell->type == ID($xnor)) gate_type = ID($_XOR_);  	log_assert(!gate_type.empty());  	for (int i = 0; i < GetSize(sig_y); i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\A", sig_a[i]); -		gate->setPort("\\B", sig_b[i]); -		gate->setPort("\\Y", sig_y[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID::A, sig_a[i]); +		gate->setPort(ID::B, sig_b[i]); +		gate->setPort(ID::Y, sig_y[i]);  	}  }  void simplemap_reduce(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	if (sig_y.size() == 0)  		return;  	if (sig_a.size() == 0) { -		if (cell->type == "$reduce_and")  module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(1, sig_y.size()))); -		if (cell->type == "$reduce_or")   module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(0, sig_y.size()))); -		if (cell->type == "$reduce_xor")  module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(0, sig_y.size()))); -		if (cell->type == "$reduce_xnor") module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(1, sig_y.size()))); -		if (cell->type == "$reduce_bool") module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(0, sig_y.size()))); +		if (cell->type == ID($reduce_and))  module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(1, sig_y.size()))); +		if (cell->type == ID($reduce_or))   module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(0, sig_y.size()))); +		if (cell->type == ID($reduce_xor))  module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(0, sig_y.size()))); +		if (cell->type == ID($reduce_xnor)) module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(1, sig_y.size()))); +		if (cell->type == ID($reduce_bool)) module->connect(RTLIL::SigSig(sig_y, RTLIL::SigSpec(0, sig_y.size())));  		return;  	} @@ -112,12 +112,12 @@ void simplemap_reduce(RTLIL::Module *module, RTLIL::Cell *cell)  		sig_y = sig_y.extract(0, 1);  	} -	std::string gate_type; -	if (cell->type == "$reduce_and")  gate_type = "$_AND_"; -	if (cell->type == "$reduce_or")   gate_type = "$_OR_"; -	if (cell->type == "$reduce_xor")  gate_type = "$_XOR_"; -	if (cell->type == "$reduce_xnor") gate_type = "$_XOR_"; -	if (cell->type == "$reduce_bool") gate_type = "$_OR_"; +	IdString gate_type; +	if (cell->type == ID($reduce_and))  gate_type = ID($_AND_); +	if (cell->type == ID($reduce_or))   gate_type = ID($_OR_); +	if (cell->type == ID($reduce_xor))  gate_type = ID($_XOR_); +	if (cell->type == ID($reduce_xnor)) gate_type = ID($_XOR_); +	if (cell->type == ID($reduce_bool)) gate_type = ID($_OR_);  	log_assert(!gate_type.empty());  	RTLIL::Cell *last_output_cell = NULL; @@ -134,22 +134,22 @@ void simplemap_reduce(RTLIL::Module *module, RTLIL::Cell *cell)  			}  			RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -			gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -			gate->setPort("\\A", sig_a[i]); -			gate->setPort("\\B", sig_a[i+1]); -			gate->setPort("\\Y", sig_t[i/2]); +			gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +			gate->setPort(ID::A, sig_a[i]); +			gate->setPort(ID::B, sig_a[i+1]); +			gate->setPort(ID::Y, sig_t[i/2]);  			last_output_cell = gate;  		}  		sig_a = sig_t;  	} -	if (cell->type == "$reduce_xnor") { +	if (cell->type == ID($reduce_xnor)) {  		RTLIL::SigSpec sig_t = module->addWire(NEW_ID); -		RTLIL::Cell *gate = module->addCell(NEW_ID, "$_NOT_"); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\A", sig_a); -		gate->setPort("\\Y", sig_t); +		RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_NOT_)); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID::A, sig_a); +		gate->setPort(ID::Y, sig_t);  		last_output_cell = gate;  		sig_a = sig_t;  	} @@ -157,7 +157,7 @@ void simplemap_reduce(RTLIL::Module *module, RTLIL::Cell *cell)  	if (last_output_cell == NULL) {  		module->connect(RTLIL::SigSig(sig_y, sig_a));  	} else { -		last_output_cell->setPort("\\Y", sig_y); +		last_output_cell->setPort(ID::Y, sig_y);  	}  } @@ -174,11 +174,11 @@ static void logic_reduce(RTLIL::Module *module, RTLIL::SigSpec &sig, RTLIL::Cell  				continue;  			} -			RTLIL::Cell *gate = module->addCell(NEW_ID, "$_OR_"); -			gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -			gate->setPort("\\A", sig[i]); -			gate->setPort("\\B", sig[i+1]); -			gate->setPort("\\Y", sig_t[i/2]); +			RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_OR_)); +			gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +			gate->setPort(ID::A, sig[i]); +			gate->setPort(ID::B, sig[i+1]); +			gate->setPort(ID::Y, sig_t[i/2]);  		}  		sig = sig_t; @@ -190,10 +190,10 @@ static void logic_reduce(RTLIL::Module *module, RTLIL::SigSpec &sig, RTLIL::Cell  void simplemap_lognot(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A);  	logic_reduce(module, sig_a, cell); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	if (sig_y.size() == 0)  		return; @@ -203,21 +203,21 @@ void simplemap_lognot(RTLIL::Module *module, RTLIL::Cell *cell)  		sig_y = sig_y.extract(0, 1);  	} -	RTLIL::Cell *gate = module->addCell(NEW_ID, "$_NOT_"); -	gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -	gate->setPort("\\A", sig_a); -	gate->setPort("\\Y", sig_y); +	RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_NOT_)); +	gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +	gate->setPort(ID::A, sig_a); +	gate->setPort(ID::Y, sig_y);  }  void simplemap_logbin(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A);  	logic_reduce(module, sig_a, cell); -	RTLIL::SigSpec sig_b = cell->getPort("\\B"); +	RTLIL::SigSpec sig_b = cell->getPort(ID::B);  	logic_reduce(module, sig_b, cell); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	if (sig_y.size() == 0)  		return; @@ -227,41 +227,41 @@ void simplemap_logbin(RTLIL::Module *module, RTLIL::Cell *cell)  		sig_y = sig_y.extract(0, 1);  	} -	std::string gate_type; -	if (cell->type == "$logic_and") gate_type = "$_AND_"; -	if (cell->type == "$logic_or")  gate_type = "$_OR_"; +	IdString gate_type; +	if (cell->type == ID($logic_and)) gate_type = ID($_AND_); +	if (cell->type == ID($logic_or))  gate_type = ID($_OR_);  	log_assert(!gate_type.empty());  	RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -	gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -	gate->setPort("\\A", sig_a); -	gate->setPort("\\B", sig_b); -	gate->setPort("\\Y", sig_y); +	gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +	gate->setPort(ID::A, sig_a); +	gate->setPort(ID::B, sig_b); +	gate->setPort(ID::Y, sig_y);  }  void simplemap_eqne(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_b = cell->getPort("\\B"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); -	bool is_signed = cell->parameters.at("\\A_SIGNED").as_bool(); -	bool is_ne = cell->type.in("$ne", "$nex"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_b = cell->getPort(ID::B); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y); +	bool is_signed = cell->parameters.at(ID(A_SIGNED)).as_bool(); +	bool is_ne = cell->type.in(ID($ne), ID($nex));  	RTLIL::SigSpec xor_out = module->addWire(NEW_ID, max(GetSize(sig_a), GetSize(sig_b)));  	RTLIL::Cell *xor_cell = module->addXor(NEW_ID, sig_a, sig_b, xor_out, is_signed); -	xor_cell->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); +	xor_cell->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src)));  	simplemap_bitop(module, xor_cell);  	module->remove(xor_cell);  	RTLIL::SigSpec reduce_out = is_ne ? sig_y : module->addWire(NEW_ID);  	RTLIL::Cell *reduce_cell = module->addReduceOr(NEW_ID, xor_out, reduce_out); -	reduce_cell->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); +	reduce_cell->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src)));  	simplemap_reduce(module, reduce_cell);  	module->remove(reduce_cell);  	if (!is_ne) {  		RTLIL::Cell *not_cell = module->addLogicNot(NEW_ID, reduce_out, sig_y); -		not_cell->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); +		not_cell->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src)));  		simplemap_lognot(module, not_cell);  		module->remove(not_cell);  	} @@ -269,65 +269,65 @@ void simplemap_eqne(RTLIL::Module *module, RTLIL::Cell *cell)  void simplemap_mux(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_b = cell->getPort("\\B"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_b = cell->getPort(ID::B); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	for (int i = 0; i < GetSize(sig_y); i++) { -		RTLIL::Cell *gate = module->addCell(NEW_ID, "$_MUX_"); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\A", sig_a[i]); -		gate->setPort("\\B", sig_b[i]); -		gate->setPort("\\S", cell->getPort("\\S")); -		gate->setPort("\\Y", sig_y[i]); +		RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_MUX_)); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID::A, sig_a[i]); +		gate->setPort(ID::B, sig_b[i]); +		gate->setPort(ID(S), cell->getPort(ID(S))); +		gate->setPort(ID::Y, sig_y[i]);  	}  }  void simplemap_tribuf(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_e = cell->getPort("\\EN"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_e = cell->getPort(ID(EN)); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	for (int i = 0; i < GetSize(sig_y); i++) { -		RTLIL::Cell *gate = module->addCell(NEW_ID, "$_TBUF_"); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\A", sig_a[i]); -		gate->setPort("\\E", sig_e); -		gate->setPort("\\Y", sig_y[i]); +		RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_TBUF_)); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID::A, sig_a[i]); +		gate->setPort(ID(E), sig_e); +		gate->setPort(ID::Y, sig_y[i]);  	}  }  void simplemap_lut(RTLIL::Module *module, RTLIL::Cell *cell)  { -	SigSpec lut_ctrl = cell->getPort("\\A"); -	SigSpec lut_data = cell->getParam("\\LUT"); -	lut_data.extend_u0(1 << cell->getParam("\\WIDTH").as_int()); +	SigSpec lut_ctrl = cell->getPort(ID::A); +	SigSpec lut_data = cell->getParam(ID(LUT)); +	lut_data.extend_u0(1 << cell->getParam(ID(WIDTH)).as_int());  	for (int idx = 0; GetSize(lut_data) > 1; idx++) {  		SigSpec sig_s = lut_ctrl[idx];  		SigSpec new_lut_data = module->addWire(NEW_ID, GetSize(lut_data)/2);  		for (int i = 0; i < GetSize(lut_data); i += 2) { -			RTLIL::Cell *gate = module->addCell(NEW_ID, "$_MUX_"); -			gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -			gate->setPort("\\A", lut_data[i]); -			gate->setPort("\\B", lut_data[i+1]); -			gate->setPort("\\S", lut_ctrl[idx]); -			gate->setPort("\\Y", new_lut_data[i/2]); +			RTLIL::Cell *gate = module->addCell(NEW_ID, ID($_MUX_)); +			gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +			gate->setPort(ID::A, lut_data[i]); +			gate->setPort(ID::B, lut_data[i+1]); +			gate->setPort(ID(S), lut_ctrl[idx]); +			gate->setPort(ID::Y, new_lut_data[i/2]);  		}  		lut_data = new_lut_data;  	} -	module->connect(cell->getPort("\\Y"), lut_data); +	module->connect(cell->getPort(ID::Y), lut_data);  }  void simplemap_sop(RTLIL::Module *module, RTLIL::Cell *cell)  { -	SigSpec ctrl = cell->getPort("\\A"); -	SigSpec table = cell->getParam("\\TABLE"); +	SigSpec ctrl = cell->getPort(ID::A); +	SigSpec table = cell->getParam(ID(TABLE)); -	int width = cell->getParam("\\WIDTH").as_int(); -	int depth = cell->getParam("\\DEPTH").as_int(); +	int width = cell->getParam(ID(WIDTH)).as_int(); +	int depth = cell->getParam(ID(DEPTH)).as_int();  	table.extend_u0(2 * width * depth);  	SigSpec products; @@ -348,213 +348,213 @@ void simplemap_sop(RTLIL::Module *module, RTLIL::Cell *cell)  		products.append(GetSize(in) > 0 ? module->Eq(NEW_ID, in, pat) : State::S1);  	} -	module->connect(cell->getPort("\\Y"), module->ReduceOr(NEW_ID, products)); +	module->connect(cell->getPort(ID::Y), module->ReduceOr(NEW_ID, products));  }  void simplemap_slice(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int offset = cell->parameters.at("\\OFFSET").as_int(); -	RTLIL::SigSpec sig_a = cell->getPort("\\A"); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	int offset = cell->parameters.at(ID(OFFSET)).as_int(); +	RTLIL::SigSpec sig_a = cell->getPort(ID::A); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	module->connect(RTLIL::SigSig(sig_y, sig_a.extract(offset, sig_y.size())));  }  void simplemap_concat(RTLIL::Module *module, RTLIL::Cell *cell)  { -	RTLIL::SigSpec sig_ab = cell->getPort("\\A"); -	sig_ab.append(cell->getPort("\\B")); -	RTLIL::SigSpec sig_y = cell->getPort("\\Y"); +	RTLIL::SigSpec sig_ab = cell->getPort(ID::A); +	sig_ab.append(cell->getPort(ID::B)); +	RTLIL::SigSpec sig_y = cell->getPort(ID::Y);  	module->connect(RTLIL::SigSig(sig_y, sig_ab));  }  void simplemap_sr(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); -	char set_pol = cell->parameters.at("\\SET_POLARITY").as_bool() ? 'P' : 'N'; -	char clr_pol = cell->parameters.at("\\CLR_POLARITY").as_bool() ? 'P' : 'N'; +	int width = cell->parameters.at(ID(WIDTH)).as_int(); +	char set_pol = cell->parameters.at(ID(SET_POLARITY)).as_bool() ? 'P' : 'N'; +	char clr_pol = cell->parameters.at(ID(CLR_POLARITY)).as_bool() ? 'P' : 'N'; -	RTLIL::SigSpec sig_s = cell->getPort("\\SET"); -	RTLIL::SigSpec sig_r = cell->getPort("\\CLR"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_s = cell->getPort(ID(SET)); +	RTLIL::SigSpec sig_r = cell->getPort(ID(CLR)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q));  	std::string gate_type = stringf("$_SR_%c%c_", set_pol, clr_pol);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\S", sig_s[i]); -		gate->setPort("\\R", sig_r[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(S), sig_s[i]); +		gate->setPort(ID(R), sig_r[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_ff(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); +	int width = cell->parameters.at(ID(WIDTH)).as_int(); -	RTLIL::SigSpec sig_d = cell->getPort("\\D"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q)); -	std::string gate_type = "$_FF_"; +	IdString gate_type = ID($_FF_);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\D", sig_d[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(D), sig_d[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_dff(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); -	char clk_pol = cell->parameters.at("\\CLK_POLARITY").as_bool() ? 'P' : 'N'; +	int width = cell->parameters.at(ID(WIDTH)).as_int(); +	char clk_pol = cell->parameters.at(ID(CLK_POLARITY)).as_bool() ? 'P' : 'N'; -	RTLIL::SigSpec sig_clk = cell->getPort("\\CLK"); -	RTLIL::SigSpec sig_d = cell->getPort("\\D"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_clk = cell->getPort(ID(CLK)); +	RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q)); -	std::string gate_type = stringf("$_DFF_%c_", clk_pol); +	IdString gate_type = stringf("$_DFF_%c_", clk_pol);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\C", sig_clk); -		gate->setPort("\\D", sig_d[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(C), sig_clk); +		gate->setPort(ID(D), sig_d[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_dffe(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); -	char clk_pol = cell->parameters.at("\\CLK_POLARITY").as_bool() ? 'P' : 'N'; -	char en_pol = cell->parameters.at("\\EN_POLARITY").as_bool() ? 'P' : 'N'; +	int width = cell->parameters.at(ID(WIDTH)).as_int(); +	char clk_pol = cell->parameters.at(ID(CLK_POLARITY)).as_bool() ? 'P' : 'N'; +	char en_pol = cell->parameters.at(ID(EN_POLARITY)).as_bool() ? 'P' : 'N'; -	RTLIL::SigSpec sig_clk = cell->getPort("\\CLK"); -	RTLIL::SigSpec sig_en = cell->getPort("\\EN"); -	RTLIL::SigSpec sig_d = cell->getPort("\\D"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_clk = cell->getPort(ID(CLK)); +	RTLIL::SigSpec sig_en = cell->getPort(ID(EN)); +	RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q)); -	std::string gate_type = stringf("$_DFFE_%c%c_", clk_pol, en_pol); +	IdString gate_type = stringf("$_DFFE_%c%c_", clk_pol, en_pol);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\C", sig_clk); -		gate->setPort("\\E", sig_en); -		gate->setPort("\\D", sig_d[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(C), sig_clk); +		gate->setPort(ID(E), sig_en); +		gate->setPort(ID(D), sig_d[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_dffsr(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); -	char clk_pol = cell->parameters.at("\\CLK_POLARITY").as_bool() ? 'P' : 'N'; -	char set_pol = cell->parameters.at("\\SET_POLARITY").as_bool() ? 'P' : 'N'; -	char clr_pol = cell->parameters.at("\\CLR_POLARITY").as_bool() ? 'P' : 'N'; +	int width = cell->parameters.at(ID(WIDTH)).as_int(); +	char clk_pol = cell->parameters.at(ID(CLK_POLARITY)).as_bool() ? 'P' : 'N'; +	char set_pol = cell->parameters.at(ID(SET_POLARITY)).as_bool() ? 'P' : 'N'; +	char clr_pol = cell->parameters.at(ID(CLR_POLARITY)).as_bool() ? 'P' : 'N'; -	RTLIL::SigSpec sig_clk = cell->getPort("\\CLK"); -	RTLIL::SigSpec sig_s = cell->getPort("\\SET"); -	RTLIL::SigSpec sig_r = cell->getPort("\\CLR"); -	RTLIL::SigSpec sig_d = cell->getPort("\\D"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_clk = cell->getPort(ID(CLK)); +	RTLIL::SigSpec sig_s = cell->getPort(ID(SET)); +	RTLIL::SigSpec sig_r = cell->getPort(ID(CLR)); +	RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q)); -	std::string gate_type = stringf("$_DFFSR_%c%c%c_", clk_pol, set_pol, clr_pol); +	IdString gate_type = stringf("$_DFFSR_%c%c%c_", clk_pol, set_pol, clr_pol);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\C", sig_clk); -		gate->setPort("\\S", sig_s[i]); -		gate->setPort("\\R", sig_r[i]); -		gate->setPort("\\D", sig_d[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(C), sig_clk); +		gate->setPort(ID(S), sig_s[i]); +		gate->setPort(ID(R), sig_r[i]); +		gate->setPort(ID(D), sig_d[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_adff(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); -	char clk_pol = cell->parameters.at("\\CLK_POLARITY").as_bool() ? 'P' : 'N'; -	char rst_pol = cell->parameters.at("\\ARST_POLARITY").as_bool() ? 'P' : 'N'; +	int width = cell->parameters.at(ID(WIDTH)).as_int(); +	char clk_pol = cell->parameters.at(ID(CLK_POLARITY)).as_bool() ? 'P' : 'N'; +	char rst_pol = cell->parameters.at(ID(ARST_POLARITY)).as_bool() ? 'P' : 'N'; -	std::vector<RTLIL::State> rst_val = cell->parameters.at("\\ARST_VALUE").bits; +	std::vector<RTLIL::State> rst_val = cell->parameters.at(ID(ARST_VALUE)).bits;  	while (int(rst_val.size()) < width)  		rst_val.push_back(RTLIL::State::S0); -	RTLIL::SigSpec sig_clk = cell->getPort("\\CLK"); -	RTLIL::SigSpec sig_rst = cell->getPort("\\ARST"); -	RTLIL::SigSpec sig_d = cell->getPort("\\D"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_clk = cell->getPort(ID(CLK)); +	RTLIL::SigSpec sig_rst = cell->getPort(ID(ARST)); +	RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q)); -	std::string gate_type_0 = stringf("$_DFF_%c%c0_", clk_pol, rst_pol); -	std::string gate_type_1 = stringf("$_DFF_%c%c1_", clk_pol, rst_pol); +	IdString gate_type_0 = stringf("$_DFF_%c%c0_", clk_pol, rst_pol); +	IdString gate_type_1 = stringf("$_DFF_%c%c1_", clk_pol, rst_pol);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, rst_val.at(i) == RTLIL::State::S1 ? gate_type_1 : gate_type_0); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\C", sig_clk); -		gate->setPort("\\R", sig_rst); -		gate->setPort("\\D", sig_d[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(C), sig_clk); +		gate->setPort(ID(R), sig_rst); +		gate->setPort(ID(D), sig_d[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_dlatch(RTLIL::Module *module, RTLIL::Cell *cell)  { -	int width = cell->parameters.at("\\WIDTH").as_int(); -	char en_pol = cell->parameters.at("\\EN_POLARITY").as_bool() ? 'P' : 'N'; +	int width = cell->parameters.at(ID(WIDTH)).as_int(); +	char en_pol = cell->parameters.at(ID(EN_POLARITY)).as_bool() ? 'P' : 'N'; -	RTLIL::SigSpec sig_en = cell->getPort("\\EN"); -	RTLIL::SigSpec sig_d = cell->getPort("\\D"); -	RTLIL::SigSpec sig_q = cell->getPort("\\Q"); +	RTLIL::SigSpec sig_en = cell->getPort(ID(EN)); +	RTLIL::SigSpec sig_d = cell->getPort(ID(D)); +	RTLIL::SigSpec sig_q = cell->getPort(ID(Q)); -	std::string gate_type = stringf("$_DLATCH_%c_", en_pol); +	IdString gate_type = stringf("$_DLATCH_%c_", en_pol);  	for (int i = 0; i < width; i++) {  		RTLIL::Cell *gate = module->addCell(NEW_ID, gate_type); -		gate->add_strpool_attribute("\\src", cell->get_strpool_attribute("\\src")); -		gate->setPort("\\E", sig_en); -		gate->setPort("\\D", sig_d[i]); -		gate->setPort("\\Q", sig_q[i]); +		gate->add_strpool_attribute(ID(src), cell->get_strpool_attribute(ID(src))); +		gate->setPort(ID(E), sig_en); +		gate->setPort(ID(D), sig_d[i]); +		gate->setPort(ID(Q), sig_q[i]);  	}  }  void simplemap_get_mappers(std::map<RTLIL::IdString, void(*)(RTLIL::Module*, RTLIL::Cell*)> &mappers)  { -	mappers["$not"]         = simplemap_not; -	mappers["$pos"]         = simplemap_pos; -	mappers["$and"]         = simplemap_bitop; -	mappers["$or"]          = simplemap_bitop; -	mappers["$xor"]         = simplemap_bitop; -	mappers["$xnor"]        = simplemap_bitop; -	mappers["$reduce_and"]  = simplemap_reduce; -	mappers["$reduce_or"]   = simplemap_reduce; -	mappers["$reduce_xor"]  = simplemap_reduce; -	mappers["$reduce_xnor"] = simplemap_reduce; -	mappers["$reduce_bool"] = simplemap_reduce; -	mappers["$logic_not"]   = simplemap_lognot; -	mappers["$logic_and"]   = simplemap_logbin; -	mappers["$logic_or"]    = simplemap_logbin; -	mappers["$eq"]          = simplemap_eqne; -	mappers["$eqx"]         = simplemap_eqne; -	mappers["$ne"]          = simplemap_eqne; -	mappers["$nex"]         = simplemap_eqne; -	mappers["$mux"]         = simplemap_mux; -	mappers["$tribuf"]      = simplemap_tribuf; -	mappers["$lut"]         = simplemap_lut; -	mappers["$sop"]         = simplemap_sop; -	mappers["$slice"]       = simplemap_slice; -	mappers["$concat"]      = simplemap_concat; -	mappers["$sr"]          = simplemap_sr; -	mappers["$ff"]          = simplemap_ff; -	mappers["$dff"]         = simplemap_dff; -	mappers["$dffe"]        = simplemap_dffe; -	mappers["$dffsr"]       = simplemap_dffsr; -	mappers["$adff"]        = simplemap_adff; -	mappers["$dlatch"]      = simplemap_dlatch; +	mappers[ID($not)]         = simplemap_not; +	mappers[ID($pos)]         = simplemap_pos; +	mappers[ID($and)]         = simplemap_bitop; +	mappers[ID($or)]          = simplemap_bitop; +	mappers[ID($xor)]         = simplemap_bitop; +	mappers[ID($xnor)]        = simplemap_bitop; +	mappers[ID($reduce_and)]  = simplemap_reduce; +	mappers[ID($reduce_or)]   = simplemap_reduce; +	mappers[ID($reduce_xor)]  = simplemap_reduce; +	mappers[ID($reduce_xnor)] = simplemap_reduce; +	mappers[ID($reduce_bool)] = simplemap_reduce; +	mappers[ID($logic_not)]   = simplemap_lognot; +	mappers[ID($logic_and)]   = simplemap_logbin; +	mappers[ID($logic_or)]    = simplemap_logbin; +	mappers[ID($eq)]          = simplemap_eqne; +	mappers[ID($eqx)]         = simplemap_eqne; +	mappers[ID($ne)]          = simplemap_eqne; +	mappers[ID($nex)]         = simplemap_eqne; +	mappers[ID($mux)]         = simplemap_mux; +	mappers[ID($tribuf)]      = simplemap_tribuf; +	mappers[ID($lut)]         = simplemap_lut; +	mappers[ID($sop)]         = simplemap_sop; +	mappers[ID($slice)]       = simplemap_slice; +	mappers[ID($concat)]      = simplemap_concat; +	mappers[ID($sr)]          = simplemap_sr; +	mappers[ID($ff)]          = simplemap_ff; +	mappers[ID($dff)]         = simplemap_dff; +	mappers[ID($dffe)]        = simplemap_dffe; +	mappers[ID($dffsr)]       = simplemap_dffsr; +	mappers[ID($adff)]        = simplemap_adff; +	mappers[ID($dlatch)]      = simplemap_dlatch;  }  void simplemap(RTLIL::Module *module, RTLIL::Cell *cell) diff --git a/passes/techmap/techmap.cc b/passes/techmap/techmap.cc index 2708b08ae..c4496f76f 100644 --- a/passes/techmap/techmap.cc +++ b/passes/techmap/techmap.cc @@ -39,20 +39,20 @@ YOSYS_NAMESPACE_END  USING_YOSYS_NAMESPACE  PRIVATE_NAMESPACE_BEGIN -void apply_prefix(std::string prefix, std::string &id) +void apply_prefix(IdString prefix, IdString &id)  {  	if (id[0] == '\\') -		id = prefix + "." + id.substr(1); +		id = stringf("%s.%s", prefix.c_str(), id.c_str()+1);  	else -		id = "$techmap" + prefix + "." + id; +		id = stringf("$techmap%s.%s", prefix.c_str(), id.c_str());  } -void apply_prefix(std::string prefix, RTLIL::SigSpec &sig, RTLIL::Module *module) +void apply_prefix(IdString prefix, RTLIL::SigSpec &sig, RTLIL::Module *module)  {  	vector<SigChunk> chunks = sig;  	for (auto &chunk : chunks)  		if (chunk.wire != NULL) { -			std::string wire_name = chunk.wire->name.str(); +			IdString wire_name = chunk.wire->name;  			apply_prefix(prefix, wire_name);  			log_assert(module->wires_.count(wire_name) > 0);  			chunk.wire = module->wires_[wire_name]; @@ -145,8 +145,8 @@ struct TechmapWorker  				record.wire = it.second;  				record.value = it.second;  				result[p].push_back(record); -				it.second->attributes["\\keep"] = RTLIL::Const(1); -				it.second->attributes["\\_techmap_special_"] = RTLIL::Const(1); +				it.second->attributes[ID::keep] = RTLIL::Const(1); +				it.second->attributes[ID(_techmap_special_)] = RTLIL::Const(1);  			}  		} @@ -175,11 +175,11 @@ struct TechmapWorker  		}  		std::string orig_cell_name; -		pool<string> extra_src_attrs = cell->get_strpool_attribute("\\src"); +		pool<string> extra_src_attrs = cell->get_strpool_attribute(ID(src));  		if (!flatten_mode) {  			for (auto &it : tpl->cells_) -				if (it.first == "\\_TECHMAP_REPLACE_") { +				if (it.first == ID(_TECHMAP_REPLACE_)) {  					orig_cell_name = cell->name.str();  					module->rename(cell, stringf("$techmap%d", autoidx++) + cell->name.str());  					break; @@ -189,16 +189,16 @@ struct TechmapWorker  		dict<IdString, IdString> memory_renames;  		for (auto &it : tpl->memories) { -			std::string m_name = it.first.str(); -			apply_prefix(cell->name.str(), m_name); +			IdString m_name = it.first; +			apply_prefix(cell->name, m_name);  			RTLIL::Memory *m = new RTLIL::Memory;  			m->name = m_name;  			m->width = it.second->width;  			m->start_offset = it.second->start_offset;  			m->size = it.second->size;  			m->attributes = it.second->attributes; -			if (m->attributes.count("\\src")) -				m->add_strpool_attribute("\\src", extra_src_attrs); +			if (m->attributes.count(ID(src))) +				m->add_strpool_attribute(ID(src), extra_src_attrs);  			module->memories[m->name] = m;  			memory_renames[it.first] = m->name;  			design->select(module, m); @@ -209,16 +209,16 @@ struct TechmapWorker  		for (auto &it : tpl->wires_) {  			if (it.second->port_id > 0)  				positional_ports[stringf("$%d", it.second->port_id)] = it.first; -			std::string w_name = it.second->name.str(); -			apply_prefix(cell->name.str(), w_name); +			IdString w_name = it.second->name; +			apply_prefix(cell->name, w_name);  			RTLIL::Wire *w = module->addWire(w_name, it.second);  			w->port_input = false;  			w->port_output = false;  			w->port_id = 0; -			if (it.second->get_bool_attribute("\\_techmap_special_")) +			if (it.second->get_bool_attribute(ID(_techmap_special_)))  				w->attributes.clear(); -			if (w->attributes.count("\\src")) -				w->add_strpool_attribute("\\src", extra_src_attrs); +			if (w->attributes.count(ID(src))) +				w->add_strpool_attribute(ID(src), extra_src_attrs);  			design->select(module, w);  		} @@ -257,18 +257,18 @@ struct TechmapWorker  			if (w->port_output && !w->port_input) {  				c.first = it.second;  				c.second = RTLIL::SigSpec(w); -				apply_prefix(cell->name.str(), c.second, module); +				apply_prefix(cell->name, c.second, module);  				extra_connect.first = c.second;  				extra_connect.second = c.first;  			} else if (!w->port_output && w->port_input) {  				c.first = RTLIL::SigSpec(w);  				c.second = it.second; -				apply_prefix(cell->name.str(), c.first, module); +				apply_prefix(cell->name, c.first, module);  				extra_connect.first = c.first;  				extra_connect.second = c.second;  			} else {  				SigSpec sig_tpl = w, sig_tpl_pf = w, sig_mod = it.second; -				apply_prefix(cell->name.str(), sig_tpl_pf, module); +				apply_prefix(cell->name, sig_tpl_pf, module);  				for (int i = 0; i < GetSize(sig_tpl) && i < GetSize(sig_mod); i++) {  					if (tpl_written_bits.count(tpl_sigmap(sig_tpl[i]))) {  						c.first.append(sig_mod[i]); @@ -320,7 +320,7 @@ struct TechmapWorker  				}  				for (auto &attr : w->attributes) { -					if (attr.first == "\\src") +					if (attr.first == ID(src))  						continue;  					module->connect(extra_connect);  					break; @@ -330,13 +330,13 @@ struct TechmapWorker  		for (auto &it : tpl->cells_)  		{ -			std::string c_name = it.second->name.str(); -			bool techmap_replace_cell = (!flatten_mode) && (c_name == "\\_TECHMAP_REPLACE_"); +			IdString c_name = it.second->name.str(); +			bool techmap_replace_cell = (!flatten_mode) && (c_name == ID(_TECHMAP_REPLACE_));  			if (techmap_replace_cell)  				c_name = orig_cell_name;  			else -				apply_prefix(cell->name.str(), c_name); +				apply_prefix(cell->name, c_name);  			RTLIL::Cell *c = module->addCell(c_name, it.second);  			design->select(module, c); @@ -345,24 +345,24 @@ struct TechmapWorker  				c->type = c->type.substr(1);  			for (auto &it2 : c->connections_) { -				apply_prefix(cell->name.str(), it2.second, module); +				apply_prefix(cell->name, it2.second, module);  				port_signal_map.apply(it2.second);  			} -			if (c->type == "$memrd" || c->type == "$memwr" || c->type == "$meminit") { -				IdString memid = c->getParam("\\MEMID").decode_string(); +			if (c->type.in(ID($memrd), ID($memwr), ID($meminit))) { +				IdString memid = c->getParam(ID(MEMID)).decode_string();  				log_assert(memory_renames.count(memid) != 0); -				c->setParam("\\MEMID", Const(memory_renames[memid].str())); +				c->setParam(ID(MEMID), Const(memory_renames[memid].str()));  			} -			if (c->type == "$mem") { -				string memid = c->getParam("\\MEMID").decode_string(); -				apply_prefix(cell->name.str(), memid); -				c->setParam("\\MEMID", Const(memid)); +			if (c->type == ID($mem)) { +				IdString memid = c->getParam(ID(MEMID)).decode_string(); +				apply_prefix(cell->name, memid); +				c->setParam(ID(MEMID), Const(memid.c_str()));  			} -			if (c->attributes.count("\\src")) -				c->add_strpool_attribute("\\src", extra_src_attrs); +			if (c->attributes.count(ID(src))) +				c->add_strpool_attribute(ID(src), extra_src_attrs);  			if (techmap_replace_cell)  				for (auto attr : cell->attributes) @@ -416,9 +416,9 @@ struct TechmapWorker  			}  			if (flatten_mode) { -				bool keepit = cell->get_bool_attribute("\\keep_hierarchy"); +				bool keepit = cell->get_bool_attribute(ID(keep_hierarchy));  				for (auto &tpl_name : celltypeMap.at(cell_type)) -					if (map->modules_[tpl_name]->get_bool_attribute("\\keep_hierarchy")) +					if (map->modules_[tpl_name]->get_bool_attribute(ID(keep_hierarchy)))  						keepit = true;  				if (keepit) {  					if (!flatten_keep_list[cell]) { @@ -484,13 +484,13 @@ struct TechmapWorker  				{  					std::string extmapper_name; -					if (tpl->get_bool_attribute("\\techmap_simplemap")) +					if (tpl->get_bool_attribute(ID(techmap_simplemap)))  						extmapper_name = "simplemap"; -					if (tpl->get_bool_attribute("\\techmap_maccmap")) +					if (tpl->get_bool_attribute(ID(techmap_maccmap)))  						extmapper_name = "maccmap"; -					if (tpl->attributes.count("\\techmap_wrap")) +					if (tpl->attributes.count(ID(techmap_wrap)))  						extmapper_name = "wrap";  					if (!extmapper_name.empty()) @@ -505,7 +505,7 @@ struct TechmapWorker  								m_name += stringf(":%s=%s", log_id(c.first), log_signal(c.second));  							if (extmapper_name == "wrap") -								m_name += ":" + sha1(tpl->attributes.at("\\techmap_wrap").decode_string()); +								m_name += ":" + sha1(tpl->attributes.at(ID(techmap_wrap)).decode_string());  							RTLIL::Design *extmapper_design = extern_mode && !in_recursion ? design : tpl->design;  							RTLIL::Module *extmapper_module = extmapper_design->module(m_name); @@ -520,7 +520,7 @@ struct TechmapWorker  								int port_counter = 1;  								for (auto &c : extmapper_cell->connections_) {  									RTLIL::Wire *w = extmapper_module->addWire(c.first, GetSize(c.second)); -									if (w->name == "\\Y" || w->name == "\\Q") +									if (w->name.in(ID::Y, ID(Q)))  										w->port_output = true;  									else  										w->port_input = true; @@ -541,14 +541,14 @@ struct TechmapWorker  								if (extmapper_name == "maccmap") {  									log("Creating %s with maccmap.\n", log_id(extmapper_module)); -									if (extmapper_cell->type != "$macc") +									if (extmapper_cell->type != ID($macc))  										log_error("The maccmap mapper can only map $macc (not %s) cells!\n", log_id(extmapper_cell->type));  									maccmap(extmapper_module, extmapper_cell);  									extmapper_module->remove(extmapper_cell);  								}  								if (extmapper_name == "wrap") { -									std::string cmd_string = tpl->attributes.at("\\techmap_wrap").decode_string(); +									std::string cmd_string = tpl->attributes.at(ID(techmap_wrap)).decode_string();  									log("Running \"%s\" on wrapper %s.\n", cmd_string.c_str(), log_id(extmapper_module));  									mkdebug.on();  									Pass::call_on_module(extmapper_design, extmapper_module, cmd_string); @@ -587,7 +587,7 @@ struct TechmapWorker  							}  							if (extmapper_name == "maccmap") { -								if (cell->type != "$macc") +								if (cell->type != ID($macc))  									log_error("The maccmap mapper can only map $macc (not %s) cells!\n", log_id(cell->type));  								maccmap(module, cell);  							} @@ -616,8 +616,8 @@ struct TechmapWorker  						continue;  					} -					if (tpl->avail_parameters.count("\\_TECHMAP_CELLTYPE_") != 0) -						parameters["\\_TECHMAP_CELLTYPE_"] = RTLIL::unescape_id(cell->type); +					if (tpl->avail_parameters.count(ID(_TECHMAP_CELLTYPE_)) != 0) +						parameters[ID(_TECHMAP_CELLTYPE_)] = RTLIL::unescape_id(cell->type);  					for (auto conn : cell->connections()) {  						if (tpl->avail_parameters.count(stringf("\\_TECHMAP_CONSTMSK_%s_", RTLIL::id2cstr(conn.first))) != 0) { @@ -656,8 +656,8 @@ struct TechmapWorker  							bits = i;  					// Increment index by one to get number of bits  					bits++; -					if (tpl->avail_parameters.count("\\_TECHMAP_BITS_CONNMAP_")) -						parameters["\\_TECHMAP_BITS_CONNMAP_"] = bits; +					if (tpl->avail_parameters.count(ID(_TECHMAP_BITS_CONNMAP_))) +						parameters[ID(_TECHMAP_BITS_CONNMAP_)] = bits;  					for (auto conn : cell->connections())  						if (tpl->avail_parameters.count(stringf("\\_TECHMAP_CONNMAP_%s_", RTLIL::id2cstr(conn.first))) != 0) { @@ -943,7 +943,8 @@ struct TechmapPass : public Pass {  		log("        instead of inlining them.\n");  		log("\n");  		log("    -max_iter <number>\n"); -		log("        only run the specified number of iterations.\n"); +		log("        only run the specified number of iterations on each module.\n"); +		log("        default: unlimited\n");  		log("\n");  		log("    -recursive\n");  		log("        instead of the iterative breadth-first algorithm use a recursive\n"); @@ -1136,8 +1137,8 @@ struct TechmapPass : public Pass {  		std::map<RTLIL::IdString, std::set<RTLIL::IdString, RTLIL::sort_by_id_str>> celltypeMap;  		for (auto &it : map->modules_) { -			if (it.second->attributes.count("\\techmap_celltype") && !it.second->attributes.at("\\techmap_celltype").bits.empty()) { -				char *p = strdup(it.second->attributes.at("\\techmap_celltype").decode_string().c_str()); +			if (it.second->attributes.count(ID(techmap_celltype)) && !it.second->attributes.at(ID(techmap_celltype)).bits.empty()) { +				char *p = strdup(it.second->attributes.at(ID(techmap_celltype)).decode_string().c_str());  				for (char *q = strtok(p, " \t\r\n"); q; q = strtok(NULL, " \t\r\n"))  					celltypeMap[RTLIL::escape_id(q)].insert(it.first);  				free(p); @@ -1157,15 +1158,16 @@ struct TechmapPass : public Pass {  			RTLIL::Module *module = *worker.module_queue.begin();  			worker.module_queue.erase(module); +			int module_max_iter = max_iter;  			bool did_something = true;  			std::set<RTLIL::Cell*> handled_cells;  			while (did_something) {  				did_something = false; -					if (worker.techmap_module(design, module, map, handled_cells, celltypeMap, false)) -						did_something = true; +				if (worker.techmap_module(design, module, map, handled_cells, celltypeMap, false)) +					did_something = true;  				if (did_something)  					module->check(); -				if (max_iter > 0 && --max_iter == 0) +				if (module_max_iter > 0 && --module_max_iter == 0)  					break;  			}  		} @@ -1222,7 +1224,7 @@ struct FlattenPass : public Pass {  		RTLIL::Module *top_mod = NULL;  		if (design->full_selection())  			for (auto mod : design->modules()) -				if (mod->get_bool_attribute("\\top")) +				if (mod->get_bool_attribute(ID(top)))  					top_mod = mod;  		std::set<RTLIL::Cell*> handled_cells; diff --git a/passes/techmap/tribuf.cc b/passes/techmap/tribuf.cc index 587cb9038..41fdc8f3d 100644 --- a/passes/techmap/tribuf.cc +++ b/passes/techmap/tribuf.cc @@ -63,38 +63,38 @@ struct TribufWorker {  		for (auto cell : module->selected_cells())  		{ -			if (cell->type == "$tribuf") -				tribuf_cells[sigmap(cell->getPort("\\Y"))].push_back(cell); +			if (cell->type == ID($tribuf)) +				tribuf_cells[sigmap(cell->getPort(ID::Y))].push_back(cell); -			if (cell->type == "$_TBUF_") -				tribuf_cells[sigmap(cell->getPort("\\Y"))].push_back(cell); +			if (cell->type == ID($_TBUF_)) +				tribuf_cells[sigmap(cell->getPort(ID::Y))].push_back(cell); -			if (cell->type.in("$mux", "$_MUX_")) +			if (cell->type.in(ID($mux), ID($_MUX_)))  			{ -				IdString en_port = cell->type == "$mux" ? "\\EN" : "\\E"; -				IdString tri_type = cell->type == "$mux" ? "$tribuf" : "$_TBUF_"; +				IdString en_port = cell->type == ID($mux) ? ID(EN) : ID(E); +				IdString tri_type = cell->type == ID($mux) ? ID($tribuf) : ID($_TBUF_); -				if (is_all_z(cell->getPort("\\A")) && is_all_z(cell->getPort("\\B"))) { +				if (is_all_z(cell->getPort(ID::A)) && is_all_z(cell->getPort(ID::B))) {  					module->remove(cell);  					continue;  				} -				if (is_all_z(cell->getPort("\\A"))) { -					cell->setPort("\\A", cell->getPort("\\B")); -					cell->setPort(en_port, cell->getPort("\\S")); -					cell->unsetPort("\\B"); -					cell->unsetPort("\\S"); +				if (is_all_z(cell->getPort(ID::A))) { +					cell->setPort(ID::A, cell->getPort(ID::B)); +					cell->setPort(en_port, cell->getPort(ID(S))); +					cell->unsetPort(ID::B); +					cell->unsetPort(ID(S));  					cell->type = tri_type; -					tribuf_cells[sigmap(cell->getPort("\\Y"))].push_back(cell); +					tribuf_cells[sigmap(cell->getPort(ID::Y))].push_back(cell);  					continue;  				} -				if (is_all_z(cell->getPort("\\B"))) { -					cell->setPort(en_port, module->Not(NEW_ID, cell->getPort("\\S"))); -					cell->unsetPort("\\B"); -					cell->unsetPort("\\S"); +				if (is_all_z(cell->getPort(ID::B))) { +					cell->setPort(en_port, module->Not(NEW_ID, cell->getPort(ID(S)))); +					cell->unsetPort(ID::B); +					cell->unsetPort(ID(S));  					cell->type = tri_type; -					tribuf_cells[sigmap(cell->getPort("\\Y"))].push_back(cell); +					tribuf_cells[sigmap(cell->getPort(ID::Y))].push_back(cell);  					continue;  				}  			} @@ -118,11 +118,11 @@ struct TribufWorker {  				SigSpec pmux_b, pmux_s;  				for (auto cell : it.second) { -					if (cell->type == "$tribuf") -						pmux_s.append(cell->getPort("\\EN")); +					if (cell->type == ID($tribuf)) +						pmux_s.append(cell->getPort(ID(EN)));  					else -						pmux_s.append(cell->getPort("\\E")); -					pmux_b.append(cell->getPort("\\A")); +						pmux_s.append(cell->getPort(ID(E))); +					pmux_b.append(cell->getPort(ID::A));  					module->remove(cell);  				} diff --git a/passes/techmap/zinit.cc b/passes/techmap/zinit.cc index 2aefc091d..ac3d4ed4a 100644 --- a/passes/techmap/zinit.cc +++ b/passes/techmap/zinit.cc @@ -62,12 +62,12 @@ struct ZinitPass : public Pass {  			for (auto wire : module->selected_wires())  			{ -				if (wire->attributes.count("\\init") == 0) +				if (wire->attributes.count(ID(init)) == 0)  					continue;  				SigSpec wirebits = sigmap(wire); -				Const initval = wire->attributes.at("\\init"); -				wire->attributes.erase("\\init"); +				Const initval = wire->attributes.at(ID(init)); +				wire->attributes.erase(ID(init));  				for (int i = 0; i < GetSize(wirebits) && i < GetSize(initval); i++)  				{ @@ -90,12 +90,12 @@ struct ZinitPass : public Pass {  			}  			pool<IdString> dff_types = { -				"$ff", "$dff", "$dffe", "$dffsr", "$adff", -				"$_FF_", "$_DFFE_NN_", "$_DFFE_NP_", "$_DFFE_PN_", "$_DFFE_PP_", -				"$_DFFSR_NNN_", "$_DFFSR_NNP_", "$_DFFSR_NPN_", "$_DFFSR_NPP_", -				"$_DFFSR_PNN_", "$_DFFSR_PNP_", "$_DFFSR_PPN_", "$_DFFSR_PPP_", -				"$_DFF_N_", "$_DFF_NN0_", "$_DFF_NN1_", "$_DFF_NP0_", "$_DFF_NP1_", -				"$_DFF_P_", "$_DFF_PN0_", "$_DFF_PN1_", "$_DFF_PP0_", "$_DFF_PP1_" +				ID($ff), ID($dff), ID($dffe), ID($dffsr), ID($adff), +				ID($_FF_), ID($_DFFE_NN_), ID($_DFFE_NP_), ID($_DFFE_PN_), ID($_DFFE_PP_), +				ID($_DFFSR_NNN_), ID($_DFFSR_NNP_), ID($_DFFSR_NPN_), ID($_DFFSR_NPP_), +				ID($_DFFSR_PNN_), ID($_DFFSR_PNP_), ID($_DFFSR_PPN_), ID($_DFFSR_PPP_), +				ID($_DFF_N_), ID($_DFF_NN0_), ID($_DFF_NN1_), ID($_DFF_NP0_), ID($_DFF_NP1_), +				ID($_DFF_P_), ID($_DFF_PN0_), ID($_DFF_PN1_), ID($_DFF_PP0_), ID($_DFF_PP1_)  			};  			for (auto cell : module->selected_cells()) @@ -103,8 +103,8 @@ struct ZinitPass : public Pass {  				if (!dff_types.count(cell->type))  					continue; -				SigSpec sig_d = sigmap(cell->getPort("\\D")); -				SigSpec sig_q = sigmap(cell->getPort("\\Q")); +				SigSpec sig_d = sigmap(cell->getPort(ID(D))); +				SigSpec sig_q = sigmap(cell->getPort(ID(Q)));  				if (GetSize(sig_d) < 1 || GetSize(sig_q) < 1)  					continue; @@ -120,14 +120,14 @@ struct ZinitPass : public Pass {  				}  				Wire *initwire = module->addWire(NEW_ID, GetSize(initval)); -				initwire->attributes["\\init"] = initval; +				initwire->attributes[ID(init)] = initval;  				for (int i = 0; i < GetSize(initwire); i++)  					if (initval.bits.at(i) == State::S1)  					{  						sig_d[i] = module->NotGate(NEW_ID, sig_d[i]);  						module->addNotGate(NEW_ID, SigSpec(initwire, i), sig_q[i]); -						initwire->attributes["\\init"].bits.at(i) = State::S0; +						initwire->attributes[ID(init)].bits.at(i) = State::S0;  					}  					else  					{ @@ -137,8 +137,8 @@ struct ZinitPass : public Pass {  				log("FF init value for cell %s (%s): %s = %s\n", log_id(cell), log_id(cell->type),  						log_signal(sig_q), log_signal(initval)); -				cell->setPort("\\D", sig_d); -				cell->setPort("\\Q", initwire); +				cell->setPort(ID(D), sig_d); +				cell->setPort(ID(Q), initwire);  			}  			for (auto &it : initbits) | 
