ARM: Apply -fmerge-all-constants to the build (saves ~1kb)

fmerge-constants appears to be the default, but while it says it will merge strings it doesn't appear to do this always, for instance for jswSymbols_InternalError_proto_str and jswSymbols_ReferenceError_proto_str which are identical.
espruino · Aug 18, 2023 · 01193e5 · 01193e5
1 parent 7aa4a4a
commit 01193e5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/ChangeLog b/ChangeLog
@@ -47,6 +47,7 @@
             nrf52: Allow disabling pairing with `NRF.setSecurity({pair:0})`
             Allow E.pipe to pipe from Strings, remove pipe's internal `position` counter (fix #2352)
             Bangle.js 1: remove OneWire and JIT as it uses too much flash space to exist alongside UTF8+Tensorflow on nRF52
+            ARM: Apply -fmerge-all-constants to the build (saves ~1kb)
 
      2v18 : Fix drawString with setClipRect with 90/270-degree rotation (fix #2343)
             Pico/Wifi: Enabled JIT compiler

diff --git a/make/common/ARM.make b/make/common/ARM.make
@@ -7,7 +7,7 @@ DEFINES += -DARM
 ifndef ARM_HAS_OWN_CMSIS # nRF5x targets do not use the shared CMSIS files.
   INCLUDE += -I$(ROOT)/targetlibs/arm
 endif
-OPTIMIZEFLAGS += -fno-common -fno-exceptions -fdata-sections -ffunction-sections
+OPTIMIZEFLAGS += -fno-common -fno-exceptions -fdata-sections -ffunction-sections -fmerge-all-constants
 
 # I've no idea why this breaks the bootloader, but it does.
 # Given we've left 10k for it, there's no real reason to enable LTO anyway.