Try CompileFast for MS.Extensions.ObjectMethodExecutor

[dadhi]

There is a shared source package used at least by ASP .NET Core:
https://github.com/aspnet/Common/blob/ff87989d893b000aac1bfef0157c92be1f04f714/shared/Microsoft.Extensions.ObjectMethodExecutor.Sources/ObjectMethodExecutor.cs

Here is the umbrella post by @shanselman
https://www.hanselman.com/blog/ASPNETCoreArchitectDavidFowlersHiddenGemsIn21.aspx

and respective tweet by @davidfowl
https://twitter.com/davidfowl/status/1004237863913906176

I may guess if FEC works, the benifits may be neglectible.
Anyway, it is a good real world case, let's check :)

[davidfowl]

Interested in the outcome. Keep me posted.

[dadhi]

Status: the await ExecuteAsync() for compiled fast async executor is crashing.
But executing without await, just returning Task works.

I have added tests 6b203fd to demonstrate the issue.

[dadhi]

Found culprit, this expression inside GetExecutorAsync:

The problem is probably due CustomAwaitableType being a struct, and then with converting the whole result to object. The good news is that expression is quite simple and can be checked in isolation to find the problem.

Without converting this particular expression above to CompileFast the results are:

Compilation:

Execution:

The compilation has a big room for improvement - here is the illustration how changing the last (problematic) expression to CompileFast adds up:

[dadhi]

/cc @davidfowl

Here is the Executor for async method with all CompileFast fixed:

Sample method: async Task<string> Foo(int a, int b) => await Task.FromResult((a + b).ToString());

Compilation - 21x faster:

Execute (blocking) - 6% faster:

ExecuteAsync - 8% faster:

[dadhi]

Sync method:
string Foo(int a, int b) => (a + b).ToString();

Compilation - 23x faster:

Execution - 19% faster: