Skip to content

[Packer] Fixed NightlyTest Failure with Tight Floorplan Constr. #3178

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
Jul 2, 2025
Merged

[Packer] Fixed NightlyTest Failure with Tight Floorplan Constr. #3178

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
32 changes: 16 additions & 16 deletions vpr/src/pack/pack.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,22 +105,6 @@ static e_packer_state get_next_packer_state(e_packer_state current_packer_state,
return e_packer_state::SUCCESS;
}

// Check if the utilizaion of any block types is over 100%
if (!fits_on_device) {
// The utilization of some block types are too high. Need to increase the
// density of the block types available.

// Check if we can turn on unrelated cluster and/or balanced block type
// utilization.
if (packer_opts.allow_unrelated_clustering == e_unrelated_clustering::AUTO && packer_opts.balance_block_type_utilization == e_balance_block_type_util::AUTO) {

// Check if they are not already on. If not, set the next state to turn them on.
if (!using_unrelated_clustering || !using_balanced_block_type_util) {
return e_packer_state::SET_UNRELATED_AND_BALANCED;
}
}
}

// Check if there are overfilled floorplan regions.
if (floorplan_regions_overfull) {
// If there are overfilled region constraints, try to use attraction
Expand Down Expand Up @@ -152,6 +136,22 @@ static e_packer_state get_next_packer_state(e_packer_state current_packer_state,
}
}

// Check if the utilizaion of any block types is over 100%
if (!fits_on_device) {
// The utilization of some block types are too high. Need to increase the
// density of the block types available.

// Check if we can turn on unrelated cluster and/or balanced block type
// utilization.
if (packer_opts.allow_unrelated_clustering == e_unrelated_clustering::AUTO && packer_opts.balance_block_type_utilization == e_balance_block_type_util::AUTO) {

// Check if they are not already on. If not, set the next state to turn them on.
if (!using_unrelated_clustering || !using_balanced_block_type_util) {
return e_packer_state::SET_UNRELATED_AND_BALANCED;
}
}
}

// Check if we can increase the target density of the overused block types.
// This is a last resort since increasing the target pin density can have
// bad affects on quality and routability.
Expand Down