Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch from issue #72: Improved use of minimization routines in PPL wrapper #83

Merged
merged 1 commit into from
Jun 26, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions ppl/ppl_poly.cc
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,6 @@ void ap_ppl_poly_minimize(ap_manager_t* man, PPL_Poly* a)
try {
/* the calls force in-place minimisation */
(void)a->p->minimized_constraints();
(void)a->p->minimized_generators();
}
CATCH_WITH_VOID(AP_FUNID_MINIMIZE);
}
Expand All @@ -227,7 +226,6 @@ void ap_ppl_poly_canonicalize(ap_manager_t* man, PPL_Poly* a)
try {
/* the calls force in-place minimisation */
(void)a->p->minimized_constraints();
(void)a->p->minimized_generators();
}
CATCH_WITH_VOID(AP_FUNID_CANONICALIZE);
}
Expand All @@ -237,7 +235,6 @@ int ap_ppl_poly_hash(ap_manager_t* man, PPL_Poly* a)
man->result.flag_exact = man->result.flag_best = true;
try {
(void)a->p->minimized_constraints();
(void)a->p->minimized_generators();
return a->p->total_memory_in_bytes();
}
CATCH_WITH_VAL(AP_FUNID_HASH,0);
Expand Down Expand Up @@ -700,7 +697,7 @@ ap_generator0_array_t ap_ppl_poly_to_generator_array(ap_manager_t* man,
try {
bool exact = true;
ap_generator0_array_t r =
ap_ppl_to_generator_array(a->p->generators(),exact);
ap_ppl_to_generator_array(a->p->minimized_generators(),exact);
if (!exact) man->result.flag_exact = man->result.flag_best = false;
return r;
}
Expand Down