caiyu/nextpnr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

kernel/sdc: Fully parse set_false_path

Browse Source
This commit is contained in:
Rowan Goemans 2024-08-12 15:41:26 +02:00
parent 7425635934
commit 69b5217963
1 changed files with 33 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
common/kernel/sdc.cc
View File

@ -376,16 +376,17 @@ struct SDCParser
TclValue cmd_set_false_path(const std::vector<TclValue> &arguments) TclValue cmd_set_false_path(const std::vector<TclValue> &arguments)
{ {
std::optional<TclValue &> from, to; NetInfo *from = nullptr;
NetInfo *to = nullptr;
for (int i = 1; i < int(arguments.size()); i++) { for (int i = 1; i < int(arguments.size()); i++) {
auto &arg = arguments.at(i); auto &arg = arguments.at(i);
if (arg.is_string) { if (arg.is_string) {
std::string s = arg.str; std::string s = arg.str;
auto &target = from; bool is_from = true;
if (s == "-to") { if (s == "-to") {
target = to; is_from = false;
} else if (s != "-from") { } else if (s != "-from") {
log_error("expecting either -to or -from to set_false_path(line %d)\n", lineno); log_error("expecting either -to or -from to set_false_path(line %d)\n", lineno);
} }
@ -396,7 +397,12 @@ struct SDCParser
log_error("expecting TclValue argument to -from (line %d)\n", lineno); log_error("expecting TclValue argument to -from (line %d)\n", lineno);
} }
for (const auto &ety : val.list) { if (val.list.size() != 1) {
log_error("Expected a single TclEntity as argument to -to/-from (line %d)\n", lineno);
}
auto &ety = val.list.at(0);
NetInfo *net = nullptr; NetInfo *net = nullptr;
if (ety.type == TclEntity::ENTITY_PIN) if (ety.type == TclEntity::ENTITY_PIN)
net = ety.get_net(ctx); net = ety.get_net(ctx);
@ -405,10 +411,24 @@ struct SDCParser
else if (ety.type == TclEntity::ENTITY_PORT) else if (ety.type == TclEntity::ENTITY_PORT)
net = ctx->ports.at(ety.name).net; net = ctx->ports.at(ety.name).net;
else else
log_error("set_false_path applies only to cells, cell pins, or IO ports (line %d)\n", lineno); log_error("set_false_path applies only to nets, cell pins, or IO ports (line %d)\n", lineno);
if (is_from) {
from = net;
} else {
to = net;
} }
} }
} }
if (from == nullptr) {
log_error("-from is required for set_false_path (line %d)\n", lineno);
} else if (to == nullptr) {
log_error("-to is required for set_false_path (line %d)\n", lineno);
}
log_info("set_false_path from: %s, to: %s\n", from->name.c_str(ctx), to->name.c_str(ctx));
return std::string{}; return std::string{};
} }