-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Span: VM side of GC reporting for Span<T>
- slow tail calls
#7263
Comments
@kouvel, any updates on this issue? |
Last I remember from talking with @jkotas about this is that this was something that is not currently an issue but could be in the future (at the time, the future was earlier in time), I don't remember the details. @jkotas, is there something here that has a potential for failure that needs to be investigated? |
Repro:
Actual result on Windows x64: prints FAILED occasionally |
Thanks, I'll take a look |
Fixes https://github.com/dotnet/coreclr/issues/9032: - Refactored by-ref-like method table walking to find offsets of by-ref pointers in siginfo.hpp/cpp - Reused that for appending GC layout when creating the copy-args helper for a slow tail call
Fixes https://github.com/dotnet/coreclr/issues/9032: - Refactored by-ref-like method table walking to find offsets of by-ref pointers in siginfo.hpp/cpp - Reused that for appending GC layout when creating the copy-args helper for a slow tail call
Fix GC reporting for slow tail call arguments of type `Span<T>` Fixes https://github.com/dotnet/coreclr/issues/9032: - Refactored by-ref-like method table walking to find offsets of by-ref pointers in siginfo.hpp/cpp - Reused that for appending GC layout when creating the copy-args helper for a slow tail call
…et#14826) Fix GC reporting for slow tail call arguments of type `Span<T>` Fixes https://github.com/dotnet/coreclr/issues/9032: - Refactored by-ref-like method table walking to find offsets of by-ref pointers in siginfo.hpp/cpp - Reused that for appending GC layout when creating the copy-args helper for a slow tail call
Fix the following places on the VM side to do proper GC reporting for
Span<T>
:StubLinkerCPU::CreateTailCallCopyArgsThunk
, or slow tailcalls have to be disabled for calls withSpan<T>
argumentsSlow tail calls may not currently be an issue, but apparently, that could change.
Forked from #7103
The text was updated successfully, but these errors were encountered: