Skip to content

Demonstrate stack overflow in compiler.#80517

Closed
CyrusNajmabadi wants to merge 6 commits intodotnet:mainfrom
CyrusNajmabadi:compilerStackOverflow
Closed

Demonstrate stack overflow in compiler.#80517
CyrusNajmabadi wants to merge 6 commits intodotnet:mainfrom
CyrusNajmabadi:compilerStackOverflow

Conversation

@CyrusNajmabadi
Copy link
Contributor

@CyrusNajmabadi CyrusNajmabadi commented Sep 30, 2025

Relates to #80518

@CyrusNajmabadi CyrusNajmabadi requested a review from a team as a code owner September 30, 2025 18:26
@jcouv jcouv marked this pull request as draft October 1, 2025 08:57
@jcouv
Copy link
Member

jcouv commented Oct 1, 2025

Marked as draft as I don't think you intend to merge this (it won't be possible due to linked bug)

@CyrusNajmabadi CyrusNajmabadi marked this pull request as ready for review October 3, 2025 12:56
@CyrusNajmabadi
Copy link
Contributor Author

@jjonescz @jcouv Test only change.

Copy link
Contributor

@AlekseyTs AlekseyTs left a comment

Choose a reason for hiding this comment

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

@CyrusNajmabadi I suggest to close this PR to avoid dealing with merges. I am currently working on the for #80518 and will make sure to include these scenarios into the test matrix. Thank you for putting them together.

@CyrusNajmabadi
Copy link
Contributor Author

Sure. That works for me. note: i included a few cases that currently work (and should likely remain that way).

Goo(new(out var x), x);
}

void Foo(C c, int x) { }
Copy link
Contributor

Choose a reason for hiding this comment

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

We generally avoid using a word "Foo" in our code base

@CyrusNajmabadi CyrusNajmabadi deleted the compilerStackOverflow branch December 1, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants