Skip to content

Commit

Permalink
fix recipe generalization application step
Browse files Browse the repository at this point in the history
  • Loading branch information
jeizenga committed Dec 14, 2022
1 parent 6bd831f commit 289bcc1
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/index_registry.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4942,8 +4942,15 @@ IndexingPlan IndexRegistry::make_plan(const IndexGroup& end_products) const {
return dep_order_of_identifier[a.first] < dep_order_of_identifier[b.first];
});

#ifdef debug_index_registry
cerr << "plan before applying generalizations:" << endl;
for (auto plan_elem : plan.steps) {
cerr << "\t" << to_string(plan_elem.first) << " " << plan_elem.second << endl;
}
#endif

// remove generalizees if we used their generalizers
set<RecipeName> plan_set(plan.steps.begin(), plan.steps.begin());
set<RecipeName> plan_set(plan.steps.begin(), plan.steps.end());
plan.steps.resize(remove_if(plan.steps.begin(), plan.steps.end(), [&](const RecipeName& recipe) {
return generalizations.count(recipe) && plan_set.count(generalizations.at(recipe));
}) - plan.steps.begin());
Expand Down

1 comment on commit 289bcc1

@adamnovak
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vg CI tests complete for branch mpmap-softclip. View the full report here.

16 tests passed, 0 tests failed and 0 tests skipped in 12286 seconds

Please sign in to comment.