Skip to content

Commit 386be5a

Browse files
[PAL] Fix vars initialization issue (flagos-ai#302)
1 parent 40e83bd commit 386be5a

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

flagcx/adaptor/tuner/tuner.cc

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
#include "tuner/tuner_util.h"
22

33
#ifdef USE_NVIDIA_ADAPTOR
4-
std::vector<EnvVar> vars = ncclTunerVars;
4+
std::vector<EnvVar> &vars = ncclTunerVars;
55
#elif USE_METAX_ADAPTOR
6-
std::vector<EnvVar> vars = mcclTunerVars;
6+
std::vector<EnvVar> &vars = mcclTunerVars;
77
#elif USE_KUNLUNXIN_ADAPTOR
8-
std::vector<EnvVar> vars = xcclTunerVars;
8+
std::vector<EnvVar> &vars = xcclTunerVars;
99
#else
10-
std::vector<EnvVar> vars = {};
10+
std::vector<EnvVar> emptyVars = {};
11+
std::vector<EnvVar> &vars = emptyVars;
1112
#endif

flagcx/adaptor/tuner/tuner_util.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,6 @@ static void safeStrCopy(char *dst, size_t dstSize, const std::string &src);
2323
extern std::vector<EnvVar> ncclTunerVars;
2424
extern std::vector<EnvVar> mcclTunerVars;
2525
extern std::vector<EnvVar> xcclTunerVars;
26-
extern std::vector<EnvVar> vars;
26+
extern std::vector<EnvVar> &vars;
2727

2828
#endif // end include guard

0 commit comments

Comments
 (0)