Re: [PATCH datacenter-manager 2/3] ui: d/rules: use thin instead of fat LTO

[Dominik Csapak <d.csapak@proxmox.com>]

On 7/7/26 4:05 PM, Fabian Grünbichler wrote:
> On July 6, 2026 1:44 pm, Dominik Csapak wrote:
>> fat LTO merges all LLVM modules into a single unit for optimizing.
>> Changing this to thin LTO increases the uncompressed binary size by 1.59
>> MiB (+11%) (compressed +192KiB/+4.6%) but save us ~160s (-40%) of (wall)
>> build time during a 'make deb'.
> 
> but what are the effects on runtime performance? the same question also
> applies to the next patch, a single codegen unit increases the amount of
> optimizations the compiler can do..


true, i didn't measure that. I'll see that i can come up with some
sensible gui benchmark to measure this and I'll report back
with the numbers..

> 
>> Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
>> ---
>>   ui/debian/rules | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/ui/debian/rules b/ui/debian/rules
>> index b3b3b078..8ffa868a 100755
>> --- a/ui/debian/rules
>> +++ b/ui/debian/rules
>> @@ -28,7 +28,7 @@ endif
>>   
>>   	$(CARGO) prepare-debian $(CURDIR)/debian/cargo_registry --link-from-system
>>   	sed -e 's/^debug = true$$/debug = false/g' debian/cargo_home/config.toml
>> -	echo "\nlto=\"fat\"" >> debian/cargo_home/config.toml
>> +	echo "\nlto=\"thin\"" >> debian/cargo_home/config.toml
>>   	echo "\nopt-level=\"s\"" >> debian/cargo_home/config.toml
>>   	echo "\ncodegen-units=1" >> debian/cargo_home/config.toml
>>   	# patch cargo_home config to use lld with wasm, otherwise the build fails
>> -- 
>> 2.47.3
>>
>>
>>
>>
>>
>>