Closed buybackoff closed 2 years ago
Thanks for raising the issue.
Looks like they changed the codegen, and now emit GC::AllocateUninitializedArray
instread of newarr
. I'll handle that.
Yep, it seems simple, but I'm not familiar with the internals.
A global switch like "EnableValidation" could be useful (like in Jil library).
What do you mean by "EnableValidation" more precisely?
When it's off, do not do any checks.
There's no check I can disable here. InlineIL does pattern matching on the IL emitted by Roslyn, and in this case Roslyn now emits different IL for method calls with params
. InlineIL has no clue what to do with this. I need to account for that.
This is caused by https://github.com/dotnet/roslyn/pull/62392, which will be reverted, but I'll still add support for this pattern to InlineIL.
Fixed in v1.7.2.
This is the same method we use in Disruptor and have discussed a lot recently.