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
[runtime] NOHANDLES for ves_icall_System_Threading_Thread_GetCurrentThread #18480
[runtime] NOHANDLES for ves_icall_System_Threading_Thread_GetCurrentThread #18480
Conversation
…hread This function (and its call to mono_handle_new) was showing up in the flamegraph for some async-heavy benchmarks
@monojenkins build deb with monolite |
@monojenkins build |
extern static Thread InitializeCurrentThread (); | ||
private extern static void InitializeCurrentThread_icall (ref Thread thread); | ||
|
||
static Thread InitializeCurrentThread () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are supposed to mark this NoInline right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm. yea. I think we want to try to replace it with an intrinsic in the near future
Follow-up forhttps://github.com/mono/pull/18480 We might want to replace calls to this with an intrinsic
Follow-up for mono#18480 We might want to replace calls to this with an intrinsic
Follow-up for #18480 We might want to replace calls to this with an intrinsic
This function (and its call to
mono_handle_new
) was showing up in theflamegraph for some async-heavy benchmarks