From 5e85757dea45899366112d09a2501e92281a3dd2 Mon Sep 17 00:00:00 2001 From: Jean-Paul Chaput Date: Fri, 7 Aug 2020 12:36:45 +0200 Subject: [PATCH] Bad asserts in Coloquinte rough_legalizer. * Bug: In Coloquinte::rough_legalizer, when running in debug mode, some assert where popping up. But, in normal mode the placer just went fine. So we assume they where too broad in their scope. Commented out. --- coloquinte/src/rough_legalizers.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/coloquinte/src/rough_legalizers.cxx b/coloquinte/src/rough_legalizers.cxx index f4fe6a06..e0d01a74 100644 --- a/coloquinte/src/rough_legalizers.cxx +++ b/coloquinte/src/rough_legalizers.cxx @@ -88,7 +88,7 @@ box region_distribution::get_box(index_t x, index_t y, index_t x_cnt, ind placement_area_.y_min + ( ((std::int64_t) (placement_area_.y_max - placement_area_.y_min)) * y ) / y_cnt, placement_area_.y_min + ( ((std::int64_t) (placement_area_.y_max - placement_area_.y_min)) * (y+1) ) / y_cnt ); - assert(not ret.empty()); + //assert(not ret.empty()); return ret; } @@ -1008,7 +1008,7 @@ std::vector region_distribution::export_sprea index_t n = R.cell_references_.size(); float_t total_capacity = static_cast(R.capacity()); box surface = static_cast >(get_box(x, y, x_regions_cnt(), y_regions_cnt())); - assert(surface.x_max > surface.x_min and surface.y_max > surface.y_min); + //assert(surface.x_max > surface.x_min and surface.y_max > surface.y_min); std::vector > x_cells, y_cells;