[core] now option conflicts in command 'build_fabric' can error out
This commit is contained in:
parent
6b92299e39
commit
b70f7fb1b6
|
@ -102,6 +102,27 @@ int build_fabric_template(T& openfpga_ctx, const Command& cmd,
|
|||
CommandOptionId opt_group_tile = cmd.option("group_tile");
|
||||
CommandOptionId opt_verbose = cmd.option("verbose");
|
||||
|
||||
/* Report conflicts with options:
|
||||
* - group tile does not support duplicate_grid_pin
|
||||
* - group tile requires compress_routing to be enabled
|
||||
*/
|
||||
if (cmd_context.option_enable(cmd, opt_group_tile)) {
|
||||
if (cmd_context.option_enable(cmd, opt_duplicate_grid_pin)) {
|
||||
VTR_LOG_ERROR(
|
||||
"Option '%s' requires options '%s' to be disabled due to a conflict!\n",
|
||||
cmd.option_name(opt_group_tile).c_str(),
|
||||
cmd.option_name(opt_duplicate_grid_pin).c_str());
|
||||
return CMD_EXEC_FATAL_ERROR;
|
||||
}
|
||||
if (!cmd_context.option_enable(cmd, opt_compress_routing)) {
|
||||
VTR_LOG_ERROR(
|
||||
"Option '%s' requires options '%s' to be enabled due to a conflict!\n",
|
||||
cmd.option_name(opt_group_tile).c_str(),
|
||||
cmd.option_name(opt_compress_routing).c_str());
|
||||
return CMD_EXEC_FATAL_ERROR;
|
||||
}
|
||||
}
|
||||
|
||||
if (true == cmd_context.option_enable(cmd, opt_compress_routing)) {
|
||||
compress_routing_hierarchy_template<T>(
|
||||
openfpga_ctx, cmd_context.option_enable(cmd, opt_verbose));
|
||||
|
|
|
@ -274,7 +274,13 @@ int build_top_module_tile_child_instances(
|
|||
if (false == frame_view) {
|
||||
/* Reserve nets to be memory efficient */
|
||||
reserve_module_manager_module_nets(module_manager, top_module);
|
||||
/* TODO: Regular nets between tiles */
|
||||
/* TODO: Regular nets between tiles
|
||||
add_top_module_nets_connect_tiles(
|
||||
module_manager, top_module, vpr_device_annotation, grids,
|
||||
tile_instance_ids, rr_graph, device_rr_gsb, compact_routing_hierarchy,
|
||||
duplicate_grid_pin);
|
||||
*/
|
||||
|
||||
/* TODO: Inter-tile direct connections */
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue