Skip to content

Commit bf3f72c

Browse files
[Pack] Fixed Precision Ambiguity
Fixed a precision ambiguity with how the inter-cluster delay is computed.
1 parent c4c8f95 commit bf3f72c

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

vpr/src/pack/pack.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -480,9 +480,7 @@ static void get_intercluster_switch_fanin_estimates(const t_arch& arch,
480480
*/
481481

482482
/* Fan-in to opin/ipin/wire switches depends on whether the architecture is unidirectional/bidirectional */
483-
// FIXME This line may have precision issues. Should 2 and 4 be floats?
484-
// This should also use parenthesis to make it more clear what each term means.
485-
(*opin_switch_fanin) = 2 * type->num_drivers / 4 * Fc_out;
483+
(*opin_switch_fanin) = 2.f * type->num_drivers / 4.f * Fc_out;
486484
(*wire_switch_fanin) = routing_arch.Fs;
487485
(*ipin_switch_fanin) = Fc_in;
488486
if (routing_arch.directionality == UNI_DIRECTIONAL) {

0 commit comments

Comments
 (0)