Skip to content

Commit 890e1c3

Browse files
committed
tuner: Fix missing config.h include
Without this, we were incorrectly including neuron headers instead of CUDA ones when HAVE_CUDA was not defined. Also fail compilation if neither definition was found rather than having a fall-through case. Signed-off-by: Raghu Raja <[email protected]>
1 parent 2adb8f5 commit 890e1c3

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

include/nccl-headers/error.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
#if HAVE_CUDA
99
#include "nccl-headers/nvidia/err.h"
10-
#else
10+
#elif HAVE_NEURON
1111
#include "nccl-headers/neuron/error.h"
1212
#endif
1313

include/nccl_ofi_tuner.h

+2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
#ifndef NCCL_OFI_TUNER_H_
22
#define NCCL_OFI_TUNER_H_
33

4+
#include "config.h"
5+
46
#include <linux/limits.h>
57
#include <float.h>
68
#include "nccl-headers/nvidia/tuner.h"

0 commit comments

Comments
 (0)