Skip to content

Commit

Permalink
Revert "SingleChipLayouter: panic if assign_region is called more…
Browse files Browse the repository at this point in the history
… than once (#20)"

This reverts commit 6c5f7d9.
  • Loading branch information
jonathanpwang authored Nov 3, 2023
1 parent 6c5f7d9 commit 3d87c8f
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions halo2_proofs/src/circuit/floor_planner/single_pass.rs
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ pub struct SingleChipLayouter<'a, F: Field, CS: Assignment<F> + 'a> {
// Stores the starting row for each region.
// Edit: modify to just one region with RegionStart(0)
// regions: Vec<RegionStart>,
// `assign_region` must only be called once.
region_assigned: bool,
/// Stores the first empty row for each column.
columns: FxHashMap<RegionColumn, usize>,
/// Stores the table fixed columns.
Expand All @@ -69,7 +67,6 @@ impl<'a, F: Field, CS: Assignment<F>> SingleChipLayouter<'a, F, CS> {
cs,
constants,
// regions: vec![],
region_assigned: false,
columns: FxHashMap::default(),
table_columns: vec![],
_marker: PhantomData,
Expand All @@ -89,10 +86,6 @@ impl<'a, F: Field, CS: Assignment<F> + 'a + SyncDeps> Layouter<F>
N: Fn() -> NR,
NR: Into<String>,
{
assert!(
!self.region_assigned,
"Only a single region can be assigned per layouter."
);
/*
let region_index = self.regions.len();
Expand Down Expand Up @@ -155,8 +148,6 @@ impl<'a, F: Field, CS: Assignment<F> + 'a + SyncDeps> Layouter<F>
}
}

self.region_assigned = true;

Ok(result)
}

Expand Down

0 comments on commit 3d87c8f

Please sign in to comment.