Skip to content

Fix no-opt inline base class call - #20639

Open
majocha wants to merge 2 commits into
dotnet:mainfrom
majocha:inline-base-call
Open

majocha wants to merge 2 commits into
dotnet:mainfrom
majocha:inline-base-call

Conversation

@majocha

@majocha majocha commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Inline members can now call inline base-class members with optimizations off while retaining debugger step-into. Uneliminated base references report FS3925 instead of an internal error.

Fixes #20614

Checklist

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

@github-actions

github-actions Bot commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

✅ Release notes checked


✅ Found changes and release notes in following paths:

Change path Release notes path Description
`src/Compiler` docs/release-notes/.FSharp.Compiler.Service/11.0.200.md

@majocha
majocha marked this pull request as ready for review September 26, 2026 12:58
@majocha
majocha requested a review from a team as a code owner September 26, 2026 12:58
@github-actions github-actions Bot added the ⚠️ Affects-Compiler-Output Tooling check: PR touches IL emission or codegen label Sep 26, 2026
@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Tooling Safety Check — Affects-Compiler-Output
Affects-Compiler-Output: Code-generation changes alter emitted compiler output.

Generated by PR Tooling Safety Check · gpt56 685K · ◷

This branch has not been deployed

No deployments
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.

Internal error when calling base class method in inlined method without optimizations

1 participant