[mono][AOT] linked away methods are emmited into final binary

Open
#118,855 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
45/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Quiet
Tech stack
csharp, wasm
Domain
compilers

Research direction

The report provides no repository file or test path. Start by reproducing the C# BDD example after IL trimming and compare the emitted WebAssembly; done means the linked-away Equals method is absent from the final binary.

Written by the indexing model from the issue text.

Description

arch-wasm area-Codegen-AOT-mono

This is a method after IL trimming

public sealed class BDD : IEquatable<BDD>
{
  [MethodImpl(MethodImplOptions.NoInlining)]
  public bool Equals(BDD other) => throw new NotSupportedException("Linked away");
}

The wasm code emmited is quite large

  (func $Wasm_Browser_Sample_BDD_Equals_BDD (;35420;) (param $var0 i32) (param $var1 i32) (param $var2 i32) (result i32)
    (local $var3 i32)
    (local $var4 i32)
    global.get $__stack_pointer
    i32.const 16
    i32.sub
    local.tee $var4
    global.set $__stack_pointer
    local.get $var4
    i32.const 3996424
    i32.load
    i32.const 21
    call $aot_wrapper_icall_mono_helper_ldstr
    local.tee $var3
    i32.store offset=12
    local.get $var4
    i32.const 33554635
    local.get $var3
    call $mono_create_corlib_exception_1
    local.tee $var3
    i32.store offset=8
    local.get $var3
    call $mini_llvmonly_throw_exception
    unreachable
  )
Dominant language
C#
Stars
18.3k
Forks
5.6k
PR merge metrics
PR metrics pending

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from dotnet/runtime

All issues in dotnet/runtime

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.