Skip to content

Commit 95265fa

Browse files
authored
Merge pull request #2 from ausimian/fix/silence-third-party-warnings
fix: silence third-party header warnings via -isystem
2 parents d8541aa + 3f55a9e commit 95265fa

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

Makefile

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,11 @@ OBJECTS := $(patsubst c_src/%.cpp,$(BUILD_DIR)/%.o,$(SOURCES))
1111

1212
# Flags
1313
CXXFLAGS := -std=c++17 -O3 -fPIC -fvisibility=hidden -Wall -Wextra
14-
CXXFLAGS += -I$(ERTS_INCLUDE_DIR) -I$(FINE_INCLUDE_DIR) -I$(MLX_INCLUDE_DIR)
14+
CXXFLAGS += -I$(ERTS_INCLUDE_DIR)
15+
# Third-party headers: use -isystem so warnings inside them (e.g. MLX's
16+
# -Wdeprecated-copy on _MLX_BFloat16) don't clutter our builds or trip
17+
# -Werror.
18+
CXXFLAGS += -isystem $(FINE_INCLUDE_DIR) -isystem $(MLX_INCLUDE_DIR)
1519

1620
LDFLAGS := -L$(MLX_LIB_DIR) -lmlx -shared
1721

0 commit comments

Comments
 (0)