Disable LTO by default.

It's too buggy and Zig will soon remove support for it anyway.

See: ziglang/zig#22230

Author: alexrp

doc/configuration/properties.md

@@ -158,7 +158,7 @@ historical reasons.
 * `FastMath` (`true`, `false`): Enable/disable certain lossy floating point
   optimizations that may not be standards-compliant. Defaults to `false`.
 * `LinkTimeOptimization` (`true`, `false`): Enable/disable link-time
-  optimization. Defaults to `true`.
+  optimization. Defaults to `false`.
 * `SymbolExports` (`Used`, `All`): Specifies whether to export all public
   symbols or only those that are needed to link successfully. This only applies
   when building executables. Defaults to `Used`.

src/sdk/build/Vezel.Zig.Sdk.Defaults.targets

@@ -58,8 +58,7 @@
         <EagerBinding Condition="'$(EagerBinding)' == ''">true</EagerBinding>
         <ExecutionModel Condition="'$(ExecutionModel)' == ''">Command</ExecutionModel>
         <FastMath Condition="'$(FastMath)' == ''">false</FastMath>
-        <!-- TODO: https://github.com/vezel-dev/zig-sdk/issues/33 -->
-        <LinkTimeOptimization Condition="'$(LinkTimeOptimization)' == ''">true</LinkTimeOptimization>
+        <LinkTimeOptimization Condition="'$(LinkTimeOptimization)' == ''">false</LinkTimeOptimization>
         <ReleaseMode Condition="'$(ReleaseMode)' == ''">Fast</ReleaseMode>
         <RelocationHardening Condition="'$(RelocationHardening)' == ''">true</RelocationHardening>
         <SymbolExports Condition="'$(SymbolExports)' == ''">Used</SymbolExports>