Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Delete FEATURE_STUBS_AS_IL define #108048

Merged
merged 7 commits into from
Sep 21, 2024
Merged

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented Sep 20, 2024

No description provided.

Copy link
Contributor

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

@jkotas jkotas force-pushed the FEATURE_STUBS_AS_IL branch from 69413fe to b810c3c Compare September 20, 2024 19:18
@@ -1235,6 +1236,49 @@ _GenericCLRToCOMCallStub@0 proc public
ret

_GenericCLRToCOMCallStub@0 endp

_GenericComCallStub@0 proc public
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ~20 instruction asm stub is a replacement for the several hundred lines of stub linker code that I am deleting. I have verified that this path is executed by (multiple) tests.

@jkotas jkotas marked this pull request as ready for review September 20, 2024 21:43
@jkotas
Copy link
Member Author

jkotas commented Sep 20, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas jkotas force-pushed the FEATURE_STUBS_AS_IL branch from e91f3f9 to e21848a Compare September 20, 2024 22:56
@jkotas
Copy link
Member Author

jkotas commented Sep 20, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas jkotas force-pushed the FEATURE_STUBS_AS_IL branch from 1bbfaaa to 3ef22c0 Compare September 20, 2024 23:29
@jkotas
Copy link
Member Author

jkotas commented Sep 20, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas
Copy link
Member Author

jkotas commented Sep 21, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas
Copy link
Member Author

jkotas commented Sep 21, 2024

/ba-g infrastructure timeout

@jkotas jkotas merged commit d456e03 into dotnet:main Sep 21, 2024
139 of 142 checks passed
@jkotas jkotas deleted the FEATURE_STUBS_AS_IL branch September 21, 2024 14:35
sirntar pushed a commit to sirntar/runtime that referenced this pull request Sep 30, 2024
* Delete FEATURE_STUBS_AS_IL define

* Delete x86-specific GenericComCallStub

* Implement GenericComCallStub
@github-actions github-actions bot locked and limited conversation to collaborators Oct 22, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants