diff options
| author |  Ayush Dubey <ayushd@google.com> | 2018-05-16 12:32:04 -0700 |
|---|---|---|
| committer |  TensorFlower Gardener <gardener@tensorflow.org> | 2018-05-16 12:34:49 -0700 |
| commit | ea3f7d1947c8a379557387b948affd918f186c41 (patch) | |
| tree | ba254d2d07012f7a92a655835144e9c360a95010 | |
| parent | d825bd42b2fce3a139c05e1412ab6a6b334ff52c (diff) | |
Remove redundant initialization of collective params.
subdiv_permutations is being resized twice in GenerateSubdivParams.
PiperOrigin-RevId: 196870781
| -rw-r--r-- | tensorflow/core/common_runtime/collective_param_resolver_local.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tensorflow/core/common_runtime/collective_param_resolver_local.cc b/tensorflow/core/common_runtime/collective_param_resolver_local.cc index 1178f8624c..8b2e0d1e0a 100644 --- a/tensorflow/core/common_runtime/collective_param_resolver_local.cc +++ b/tensorflow/core/common_runtime/collective_param_resolver_local.cc @@ -318,9 +318,6 @@ void SortDevicesAndTasks(CollectiveParams* cp) { // ring order implicit in the device order. void GenerateSubdivPerms(const string& device, int source_rank, CollectiveParams* cp) { - CHECK_GT(cp->instance.impl_details.subdiv_offsets.size(), 0); - cp->instance.impl_details.subdiv_permutations.resize( - cp->instance.impl_details.subdiv_offsets.size()); // Each subdiv permutation is a ring formed by rotating each // single-task subsequence of devices by an offset. This makes most // sense when each task has the same number of devices but we can't |