Skip to content

Improve implied argument names#20001

Open
kerams wants to merge 2 commits into
dotnet:mainfrom
kerams:args
Open

Improve implied argument names#20001
kerams wants to merge 2 commits into
dotnet:mainfrom
kerams:args

Conversation

@kerams

@kerams kerams commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Description

Continuation of #15277, filling a remaining gap.

type Combine = delegate of first: int * second: int * third: int * fourth: int -> int

[<System.Runtime.CompilerServices.MethodImpl(System.Runtime.CompilerServices.MethodImplOptions.NoInlining)>]
let f a b c d e g = a + b + c + d + e + g

let test1 = Combine(f 1 2)

Before

internal static class test1@6
{
    internal static int Invoke(int delegateArg0, int delegateArg1, int delegateArg2, int delegateArg3)
    {
        return f(1, 2, delegateArg0, delegateArg1, delegateArg2, delegateArg3);
    }
}

After

internal static class test1@6
{
    internal static int Invoke(int first, int second, int third, int fourth)
    {
        return f(1, 2, first, second, third, fourth);
    }
}

Checklist

  • Test cases added
  • Performance benchmarks added in case of performance changes
  • Release notes entry updated

@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

❗ Release notes required

You can open this PR in browser to add release notes: open in github.dev


✅ Found changes and release notes in following paths:

Change path Release notes path Description
src/Compiler docs/release-notes/.FSharp.Compiler.Service/11.0.100.md
LanguageFeatures.fsi docs/release-notes/.Language/preview.md

@github-actions github-actions Bot added the ⚠️ Affects-Compiler-Output Tooling check: PR touches IL emission or codegen label Jun 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Tooling Safety Check — Affects-Compiler-Output
Affects-Compiler-Output: modifies MethodCalls.fs changing emitted delegate argument names in IL

Generated by PR Tooling Safety Check · opus46 4.8M ·

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚠️ Affects-Compiler-Output Tooling check: PR touches IL emission or codegen

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

1 participant