Skip to content

Fix wrong [<TailCall>] warning with unit-returning match expression - #17637

Merged
vzarytovskii merged 4 commits into
dotnet:mainfrom
dawedawe:fix_17604
Aug 30, 2024
Merged

Fix wrong [<TailCall>] warning with unit-returning match expression#17637
vzarytovskii merged 4 commits into
dotnet:mainfrom
dawedawe:fix_17604

Conversation

@dawedawe

Copy link
Copy Markdown
Contributor

Description

Fixes#17604

I think we missed the handling for rec functions that are just unit-returning match expressions.

Checklist

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

@dawedawe
dawedawe requested a review from a team as a code ownerAugust 30, 2024 08:57
@github-actions

github-actionsBot commented Aug 30, 2024

Copy link
Copy Markdown
Contributor

❗ Release notes required


✅ Found changes and release notes in following paths:

Change pathRelease notes pathDescription
src/Compilerdocs/release-notes/.FSharp.Compiler.Service/9.0.100.md

@psfinakipsfinaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for a quick fix Dawe!

@vzarytovskiivzarytovskii left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks

@vzarytovskii
vzarytovskii enabled auto-merge (squash) August 30, 2024 11:11
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

TailCall attribute incorrectly warning

4 participants

@dawedawe@vzarytovskii@psfinaki@edgarfgp