-
Notifications
You must be signed in to change notification settings - Fork 783
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Moved CodeGen/EmittedIL/TailCalls over to NUnit #7386
Conversation
@KevinRansom can you please suggest why local variable name can be omitted after ILdasm? |
@KevinRansom any thoughts? |
Oh the il differences are because of optimization. The change is from building debug to building release. I believe I fixed this with this PR: |
@KevinRansom can you please share the link to the PR that you mentioned? Upd: If I got it right CompilerAssert.CompileLibraryAndVerifyILWithOptions [|"-g"; "--optimize-"; "--tailcalls+"|] but for me it does not affect |
Thanks for this. |
Thank you for merging this!
|
* TailCalls test migration * fix the diff in IL * fix for netcore
part of #7075
The only strange thing is that IL for
omit name of local variable
x
...in bsl it was
for me it looks like
full difference from bsl IL is in the commit 1703306