From 3571f8ca0b9a6f8757927f3ba71581cd306edad0 Mon Sep 17 00:00:00 2001 From: Matthew Whitlock Date: Mon, 10 Nov 2025 14:18:53 -0600 Subject: [PATCH] Always populate failed procs in comms' groups Signed-off-by: Matthew Whitlock --- ompi/communicator/ft/comm_ft.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/ompi/communicator/ft/comm_ft.c b/ompi/communicator/ft/comm_ft.c index bd2627c778c..429d574bbed 100644 --- a/ompi/communicator/ft/comm_ft.c +++ b/ompi/communicator/ft/comm_ft.c @@ -792,13 +792,12 @@ bool ompi_comm_is_proc_active(ompi_communicator_t *comm, int peer_id, bool remot int ompi_comm_set_rank_failed(ompi_communicator_t *comm, int peer_id, bool remote) { -#if OPAL_ENABLE_DEBUG /* populate the proc in the comm's group array so that it is not a sentinel and can be read as failed */ - ompi_proc_t *ompi_proc = ompi_group_get_proc_ptr((remote ? comm->c_remote_group : comm->c_local_group), - peer_id, true); + ompi_proc_t *ompi_proc __opal_attribute_unused__; + ompi_proc = ompi_group_get_proc_ptr((remote ? comm->c_remote_group : comm->c_local_group), + peer_id, true); assert(NULL != ompi_proc); -#endif /* Disable ANY_SOURCE */ comm->any_source_enabled = false;