Skip to content

Commit c18a089

Browse files
authored
Merge pull request #1439 from TeamCOMPAS/fix_buid_warnings
fix: add -Wno-vla-extension (note, -Wno-vla-cxx-extension might be needed in the future)
2 parents e47dcad + 88d4832 commit c18a089

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/Makefile

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,10 @@ ifneq ($(filter fast staticfast,$(MAKECMDGOALS)),)
9090
OPTFLAGS += $(OPTFLAGS_SPECIFIC)
9191
endif
9292

93-
CXXFLAGS := -std=c++17 -Wall $(OPTFLAGS) $(CXXFLAGS_SPECIFIC)
93+
# might need -Wno-vla-cxx-extension for clang>18?
94+
EXTRA_WARN_SUPPRESS := -Wno-vla-extension
95+
96+
CXXFLAGS := -std=c++17 -Wall $(OPTFLAGS) $(CXXFLAGS_SPECIFIC) $(EXTRA_WARN_SUPPRESS)
9497
ICFLAGS := -I$(GSLINCDIR) -I$(BOOSTINCDIR) -I$(HDF5INCDIR) -I.
9598

9699
LIBS := -lm -lz -ldl -lpthread
@@ -130,4 +133,4 @@ $(ODIR)/%.o: %.cpp
130133
# Clean-up rule
131134
clean:
132135
@echo "Removing generated files..."
133-
@rm -rf $(ODIR) $(BDIR) COMPAS
136+
@rm -rf $(ODIR) $(BDIR) COMPAS

0 commit comments

Comments
 (0)