Added a temporary comment about about the MPI v7 requirement for MPI_…

…Iallreduce call on Windows. This comment will be removed in a few weeks.
roy881020 · Jan 18, 2016 · 3d87c90 · 3d87c90
1 parent e37fede
commit 3d87c90
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/Source/SGDLib/SimpleDistGradAggregator.h b/Source/SGDLib/SimpleDistGradAggregator.h
@@ -278,6 +278,7 @@ namespace Microsoft { namespace MSR { namespace CNTK {
                     reductionBuffer = m_intermediateCPUBuffers[i].get();
                 }
 
+                // On Windows this async MPI_Iallreduce call requires MS MPI v7 or higher to be installed
                 MPI_Iallreduce(MPI_IN_PLACE, reductionBuffer, gradients[i]->GetNumElements(), MPIWrapper::GetDataType(reductionBuffer), MPI_SUM, m_mpi->Communicator(), &allReduceRequests[i]) || MpiFail("MPI_Iallreduce");
             }