Track last index in parameter marshalling#87846
Track last index in parameter marshalling#87846jtschuster wants to merge 14 commits intodotnet:mainfrom
Conversation
|
Tagging subscribers to this area: @dotnet/interop-contrib Issue DetailsTracks the last index marshalled in the interop generators to ensure we don't cleanup any unmarshalled elements. To determine if the We'll need more testing around the ComInterfaceGenerator, but I would like to get this into preview 6 for LibraryImportGenerator.
|
jkoritzinsky
left a comment
There was a problem hiding this comment.
As long as we don't generate invalid code for GeneratedComInterface, I'm supportive of getting this in for P6
|
It looks like the unit tests all timed out in CI, but only for the LibraryImportGenerator unit tests. Very odd. |
|
Looks like this probably does hurt perf significantly enough to cause some timeouts. I made #88060 as an alternative. |
|
#88060 was accepted instead of this one |
Tracks the last index marshalled in the interop generators to ensure we don't cleanup any unmarshalled elements.
To determine if the
lastElementMarshalledvariable is used, we need to generate the cleanup stage, which can be expensive. We also could generate_ = __param_lastIndexMarshalledif there is no cleanup so that there is no warning for an unused variable, but we don't need to generate the cleanup stage twice.We'll need more testing around the ComInterfaceGenerator, but I would like to get this into preview 6 for LibraryImportGenerator.