From 0fae450b9e46a21a5bc5b3bfef7b76763bbcd0d9 Mon Sep 17 00:00:00 2001 From: Nicholas Sielicki Date: Tue, 13 Aug 2024 13:02:08 -0700 Subject: [PATCH] platform-aws: align declaration and init order When using designated initializer lists, under C++ only, a warning will be emitted if the declaration order does not match the initialization order if any fields are missing in the initializer list. Reorder these structs to initialize all fields in their declaration order. stack-info: PR: https://github.com/aws/aws-ofi-nccl/pull/557, branch: aws-nslick/stack/4 Signed-off-by: Nicholas Sielicki --- src/platform-aws.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/platform-aws.c b/src/platform-aws.c index b8d6fa80a..3c06caa1e 100644 --- a/src/platform-aws.c +++ b/src/platform-aws.c @@ -95,16 +95,16 @@ struct ec2_platform_data { }, { .name = "trn1.32xlarge", - .default_protocol = "RDMA", .gdr_required = true, .net_flush_required = true, + .default_protocol = "RDMA", .domain_per_thread = 1, }, { .name = "trn1n.32xlarge", - .default_protocol = "RDMA", .gdr_required = true, .net_flush_required = true, + .default_protocol = "RDMA", .domain_per_thread = 1, } };