[c10d] Fix link order for building C++ tests (#8889)

pietern · teng-li · commit af741dc2fd87 · 2018-06-25T23:59:32.000-07:00
List dependency on gloo_cuda before dependency on gloo such that
unresolved symbols in gloo_cuda are correctly resolved (since the linker
resolves from left to right).

This fixes building c10d C++ tests on GCC 4.8.
diff --git a/torch/lib/c10d/CMakeLists.txt b/torch/lib/c10d/CMakeLists.txt
@@ -77,8 +77,8 @@ set(C10D_SRCS
 
 set(C10D_LIBS
   caffe2_gpu
-  ${Gloo_NATIVE_LIBRARY}
   ${Gloo_LIBRARY}
+  ${Gloo_NATIVE_LIBRARY}
   )
 
 if(DISTRIBUTED_NCCL_FOUND)

Original file line number	Diff line number	Diff line change
`@@ -77,8 +77,8 @@ set(C10D_SRCS`
`77`	`77`
`78`	`78`	`set(C10D_LIBS`
`79`	`79`	`caffe2_gpu`
`80`		`- ${Gloo_NATIVE_LIBRARY}`
`81`	`80`	`${Gloo_LIBRARY}`
	`81`	`+ ${Gloo_NATIVE_LIBRARY}`
`82`	`82`	`)`
`83`	`83`
`84`	`84`	`if(DISTRIBUTED_NCCL_FOUND)`