Skip to content

Commit

Permalink
Update Clang warning flags for main build
Browse files Browse the repository at this point in the history
This enables many more warnings for Clang when building explicitly with `Toolchain=clang`.
  • Loading branch information
DanRStevens committed Mar 9, 2025
1 parent 45582c9 commit 3881217
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions makefile
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,11 @@ gccPkgConfig := $(PkgConfig)
gccTARGET_OS := $(TARGET_OS)

clangCXX := clang++
clangWarnFlags := $(WarnFlags)
clangWarnNotInterested := -Wno-c++98-compat-pedantic -Wno-pre-c++17-compat
clangWarnAllowed := -Wno-padded
clangWarnKnown := -Wno-global-constructors -Wno-exit-time-destructors -Wno-unused-member-function
clangWarnShow := -Weverything $(clangWarnNotInterested)
clangWarnFlags := $(clangWarnShow) $(clangWarnAllowed) $(clangWarnKnown)
clangPkgConfig := $(PkgConfig)
clangTARGET_OS := $(TARGET_OS)

Expand Down Expand Up @@ -211,12 +215,6 @@ $(PACKAGE_NAME): $(OUTPUT) $(shell find $(SRCDIR) -name '*.h')

## Linting ##

clangWarnNotInterested := -Wno-c++98-compat-pedantic -Wno-pre-c++17-compat
clangWarnAllowed := -Wno-padded
clangWarnKnown := -Wno-global-constructors -Wno-exit-time-destructors -Wno-unused-member-function
clangWarnShow := -Weverything $(clangWarnNotInterested)
clangWarnFlags := $(clangWarnShow) $(clangWarnAllowed) $(clangWarnKnown)

.PHONY: show-warnings
show-warnings:
@$(MAKE) clean-all > /dev/null
Expand Down

0 comments on commit 3881217

Please sign in to comment.