Skip to content

Commit

Permalink
Enable WasmStripILAfterAOT by default (dotnet#94682)
Browse files Browse the repository at this point in the history
  • Loading branch information
fanyang-mono authored Nov 14, 2023
1 parent d1bb2ea commit 735e5ed
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions src/mono/wasm/Wasm.Build.Tests/Blazor/BuildPublishTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -175,8 +175,8 @@ void AssertResourcesDlls(string basePath)
}

[Theory]
[InlineData("", false)] // Default case
[InlineData("true", true)] // the other case
[InlineData("", true)] // Default case
[InlineData("false", false)] // the other case
public async Task Test_WasmStripILAfterAOT(string stripILAfterAOT, bool expectILStripping)
{
string config = "Release";
Expand Down
4 changes: 2 additions & 2 deletions src/mono/wasm/Wasm.Build.Tests/Templates/WasmTemplateTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -493,8 +493,8 @@ public void BuildAndRunForDifferentOutputPaths(string config, bool appendRID, bo
}

[Theory]
[InlineData("", false)] // Default case
[InlineData("true", true)] // the other case
[InlineData("", true)] // Default case
[InlineData("false", false)] // the other case
public void Test_WasmStripILAfterAOT(string stripILAfterAOT, bool expectILStripping)
{
string config = "Release";
Expand Down
4 changes: 2 additions & 2 deletions src/mono/wasm/build/WasmApp.targets
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@
- AppBundle/_framework contains generated files (dlls, runtime scripts, icu)
- AppBundle/_content contains web files from nuget packages (css, js, etc)
- $(WasmStripILAfterAOT) - Set to true to enable trimming away AOT compiled methods body (IL code)
Defaults to false.
Defaults to true.
Public items:
- @(WasmExtraFilesToDeploy) - Files to copy to $(WasmAppDir).
Expand Down Expand Up @@ -150,7 +150,7 @@
<WasmAssemblyExtension Condition="'$(WasmEnableWebcil)' == 'true'">.wasm</WasmAssemblyExtension>
<WasmAssemblyExtension Condition="'$(WasmEnableWebcil)' != 'true'">.dll</WasmAssemblyExtension>

<WasmStripILAfterAOT Condition="'$(WasmStripILAfterAOT)' == ''">false</WasmStripILAfterAOT>
<WasmStripILAfterAOT Condition="'$(WasmStripILAfterAOT)' == ''">true</WasmStripILAfterAOT>

<WasmRuntimeAssetsLocation Condition="'$(WasmRuntimeAssetsLocation)' == ''">_framework</WasmRuntimeAssetsLocation>
</PropertyGroup>
Expand Down

0 comments on commit 735e5ed

Please sign in to comment.