Merge pull request #3813 from vgteam/mpmap-softclip

Fix the application step for recipe generalizations
vgteam · Dec 15, 2022 · 2bc7439 · 2bc7439 · adamnovak · Dec 15, 2022
2 parents 62cab6c + 289bcc1
commit 2bc7439
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/index_registry.cpp b/src/index_registry.cpp
@@ -4953,8 +4953,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());