Skip to content

Commit

Permalink
Recompute BDD after scheduling in benchmark_main
Browse files Browse the repository at this point in the history
Scheduling can change the function and add new nodes which would not be present in the older BDD so we need to recompute it.

PiperOrigin-RevId: 610913716
  • Loading branch information
allight authored and copybara-github committed Feb 28, 2024
1 parent 3de87ec commit 6213ac5
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion xls/tools/benchmark_main.cc
Original file line number Diff line number Diff line change
Expand Up @@ -793,8 +793,12 @@ absl::Status RealMain(std::string_view path) {
'\n'))
.size());

// Scheduling can change the nodes in f slightly so we need to recompute the
// bdd.
BddQueryEngine sched_qe(BddFunction::kDefaultPathLimit);
XLS_RETURN_IF_ERROR(sched_qe.Populate(f).status());
XLS_RETURN_IF_ERROR(PrintScheduleInfo(
f, schedules, query_engine, delay_estimator,
f, schedules, sched_qe, delay_estimator,
scheduling_options_flags_proto.has_clock_period_ps()
? std::make_optional(
scheduling_options_flags_proto.clock_period_ps())
Expand Down

0 comments on commit 6213ac5

Please sign in to comment.