fix: topo: delete impossible break statement

The check immediately prior is that userdata is null, and then the first
thing we do after that is check if it is null and attempt to break out
of the scope we just entered. clang-sa complains that this is dead code.

Delete the dead code.

Signed-off-by: Nicholas Sielicki <nslick@amazon.com>

src/nccl_ofi_topo.c

@@ -435,9 +435,6 @@ static int set_userdata_to_root(hwloc_obj_t node,
 	/* Walk upwards to the root */
 	while (node) {
 		if (!node->userdata) {
-			/* Abort when a node is reached that already stores user data */
-			if (node->userdata) break;
-
 			user_data = nccl_ofi_get_user_data(data_iter);
 			if (!user_data) {
 				NCCL_OFI_WARN("Failed to access user data of data_iter");