simulator: fix missing Werror option + eliminate lib duplicate

configure:
Ensure Werror option doesn't get lost when simcc resets GCCOPTS

uisimulator.make:
Remove ignored duplicate SIMLIB on Darwin

codecs.make:
Eliminate some redundancy

Change-Id: Ieee6f677fd22666cb58aa6fe53eabdc0b0f8c190
This commit is contained in:
Christian Soffke 2024-12-18 13:08:34 +01:00
parent 0bb79dd847
commit a86b1999d0
3 changed files with 8 additions and 8 deletions

View file

@ -26,16 +26,15 @@ CODECFLAGS := $(CFLAGS) $(RBCODEC_CFLAGS) -fstrict-aliasing \
-I$(RBCODECLIB_DIR)/codecs -I$(RBCODECLIB_DIR)/codecs/lib -DCODEC
ifdef APP_TYPE
ifeq ($(UNAME), Darwin)
CODECLDFLAGS = $(SHARED_LDFLAGS) -Wl,-map,$(CODECDIR)/$*.map
else
CODECLDFLAGS = $(SHARED_LDFLAGS) -Wl,--gc-sections -Wl,-Map,$(CODECDIR)/$*.map
CODECLDFLAGS = $(SHARED_LDFLAGS)
ifneq ($(UNAME), Darwin)
CODECLDFLAGS += -Wl,--gc-sections
endif
CODECFLAGS += $(SHARED_CFLAGS) # <-- from Makefile
else
CODECLDFLAGS = -T$(CODECLINK_LDS) -Wl,--gc-sections -Wl,-Map,$(CODECDIR)/$*.map
CODECLDFLAGS = -T$(CODECLINK_LDS) -Wl,--gc-sections
endif
CODECLDFLAGS += $(GLOBAL_LDOPTS)
CODECLDFLAGS += -Wl,$(LDMAP_OPT),$(CODECDIR)/$*.map $(GLOBAL_LDOPTS)
ifdef USE_LTO
CODECLDFLAGS += -fno-builtin -ffreestanding

1
tools/configure vendored
View file

@ -1101,6 +1101,7 @@ S(m)all C lib, Logf to Ser(i)al port, LTO Build(X), (E)rror on warnings"
[Ee])
echo "Treating all warnings as errors"
GCCOPTS="$GCCOPTS -Werror"
CCOPTS="$CCOPTS -Werror"
;;
"") # Match enter press when finished with advanced options
cont=0

View file

@ -32,10 +32,10 @@ $(SIMLIB): $$(SIMOBJ) $(UIBMP)
$(BUILDDIR)/$(BINARY): $$(OBJ) $(FIRMLIB) $(VOICESPEEXLIB) $(CORE_LIBS) $(SIMLIB)
ifeq ($(UNAME), Darwin)
$(call PRINTS,LD $(BINARY))$(CC) -o $@ $^ $(SIMLIB) $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,-map,$(BUILDDIR)/rockbox.map
$(call PRINTS,LD $(BINARY))$(CC) -o $@ $^ $(LDOPTS) $(GLOBAL_LDOPTS) -Wl,$(LDMAP_OPT),$(BUILDDIR)/rockbox.map
else
$(call PRINTS,LD $(BINARY))$(CC) -o $@ -Wl,--start-group $^ -Wl,--end-group $(LDOPTS) $(GLOBAL_LDOPTS) \
-Wl,-Map,$(BUILDDIR)/rockbox.map
-Wl,$(LDMAP_OPT),$(BUILDDIR)/rockbox.map
endif
$(SILENT)$(call objcopy,$@,$@)