From 81dbbea1c86efe311388ea57c370edbea1a2127f Mon Sep 17 00:00:00 2001 From: Ayke van Laethem Date: Fri, 4 Nov 2022 13:07:20 +0100 Subject: [PATCH] esp: use ThinLTO for Xtensa This is now possible because we're using the LLVM linker. It results in some very minor code size reductions. The main benefit however is consistency: eventually, all targets will support ThinLTO at which point we can remove support for GNU linkers and simplify the compiler. --- compileopts/config.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/compileopts/config.go b/compileopts/config.go index 144cfd04..657dbd09 100644 --- a/compileopts/config.go +++ b/compileopts/config.go @@ -200,7 +200,7 @@ func (c *Config) UseThinLTO() bool { // wasm-ld doesn't seem to support ThinLTO yet. return false } - if parts[0] == "avr" || parts[0] == "xtensa" { + if parts[0] == "avr" { // These use external (GNU) linkers which might perhaps support ThinLTO // through a plugin, but it's too much hassle to set up. return false