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

Merge pull request #990 from YosysHQ/gatecat/fix-988

Browse Source 
Don't assert on mixed domain paths in report
This commit is contained in:
myrtle 2022-05-22 14:31:55 +01:00 committed by GitHub
commit 48f9daf867
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
common/kernel/report.cc
View File

@ -131,11 +131,6 @@ static Json::array report_detailed_net_timings(const Context *ctx)
Json::array endpointsJson; Json::array endpointsJson;
for (const auto &sink_timing : it.second) { for (const auto &sink_timing : it.second) {
// FIXME: Is it possible that there are multiple different start
// events for a single net? It has a single driver
NPNR_ASSERT(sink_timing.clock_pair.start == start);
auto endpointJson = Json::object({{"cell", sink_timing.cell_port.first.c_str(ctx)}, auto endpointJson = Json::object({{"cell", sink_timing.cell_port.first.c_str(ctx)},
{"port", sink_timing.cell_port.second.c_str(ctx)}, {"port", sink_timing.cell_port.second.c_str(ctx)},
{"event", clock_event_name(ctx, sink_timing.clock_pair.end)}, {"event", clock_event_name(ctx, sink_timing.clock_pair.end)},