Skip to content

Commit 877adb2

Browse files
committed
Removing dead code pertaining to the former USER paramter of the option --fix_pins
1 parent 8f11883 commit 877adb2

File tree

4 files changed

+3
-14
lines changed

4 files changed

+3
-14
lines changed

vpr/src/base/CheckSetup.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,6 @@ void CheckSetup(const t_packer_opts& PackerOpts,
3838
"Timing analysis must be enabled for timing-driven placement.\n");
3939
}
4040

41-
if (!PlacerOpts.doPlacement && (USER == PlacerOpts.pad_loc_type)) {
42-
VPR_FATAL_ERROR(VPR_ERROR_OTHER,
43-
"A pad location file requires that placement is enabled.\n");
44-
}
45-
4641
if (!PlacerOpts.doPlacement && (LOCKED == PlacerOpts.block_loc_type)) {
4742
VPR_FATAL_ERROR(VPR_ERROR_OTHER,
4843
"A block location file requires that placement is enabled.\n");

vpr/src/base/ShowSetup.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -509,9 +509,6 @@ static void ShowPlacerOpts(const t_placer_opts& PlacerOpts,
509509
case RANDOM:
510510
VTR_LOG("RANDOM\n");
511511
break;
512-
case USER:
513-
VTR_LOG("USER '%s'\n", PlacerOpts.pad_loc_file.c_str());
514-
break;
515512
default:
516513
VPR_FATAL_ERROR(VPR_ERROR_UNKNOWN, "Unknown I/O pad location type\n");
517514
}

vpr/src/base/read_options.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2217,8 +2217,7 @@ void set_conditional_defaults(t_options& args) {
22172217

22182218
args.pad_loc_file.set("", Provenance::SPECIFIED);
22192219
} else {
2220-
args.pad_loc_type.set(USER, Provenance::INFERRED);
2221-
VTR_ASSERT(!args.pad_loc_file.value().empty());
2220+
VPR_FATAL_ERROR(VPR_ERROR_UNKNOWN, "Unknown I/O pad location type\n");
22222221
}
22232222

22242223
//Are the blocks locked to locations given by a constraints file?

vpr/src/base/vpr_types.h

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -504,13 +504,11 @@ enum pfreq {
504504
};
505505

506506
/**
507-
* @brief Are the pads free to be moved, locked in a random configuration, or
508-
* locked in user-specified positions?
507+
* @brief Are the pads free to be moved or locked in a random configuration?
509508
*/
510509
enum e_pad_loc_type {
511510
FREE,
512-
RANDOM,
513-
USER
511+
RANDOM
514512
};
515513

516514
/*Are the blocks not locked (free to be moved) or locked in user-specified positions?*/

0 commit comments

Comments
 (0)