Skip to content

Convert JitHelper JIT_StrCns to QCall - #113674

Merged
AaronRobinsonMSFT merged 3 commits into
dotnet:mainfrom
AaronRobinsonMSFT:hmf_strcns
Mar 19, 2025
Merged

AaronRobinsonMSFT merged 3 commits into
dotnet:mainfrom
AaronRobinsonMSFT:hmf_strcns

Conversation

@AaronRobinsonMSFT

Copy link
Copy Markdown
Member

Remove Helper Method Frame (HMF) by converting to QCall.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR converts the JitHelper JIT_StrCns to a QCall implementation to remove the Helper Method Frame (HMF).

  • Added a new QCall method (StrCnsInternal) using LibraryImport with QCall entry point.
  • Introduced an internal StrCns method that calls StrCnsInternal and converts the returned pointer to a string.
Files not reviewed (11)
  • src/coreclr/inc/jithelpers.h: Language not supported
  • src/coreclr/vm/appdomainnative.cpp: Language not supported
  • src/coreclr/vm/appdomainnative.hpp: Language not supported
  • src/coreclr/vm/ceeload.cpp: Language not supported
  • src/coreclr/vm/ceeload.h: Language not supported
  • src/coreclr/vm/corelib.h: Language not supported
  • src/coreclr/vm/jithelpers.cpp: Language not supported
  • src/coreclr/vm/jitinterface.cpp: Language not supported
  • src/coreclr/vm/jitinterface.h: Language not supported
  • src/coreclr/vm/metasig.h: Language not supported
  • src/coreclr/vm/qcallentrypoints.cpp: Language not supported

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

@jkotas jkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM otherwise

@AaronRobinsonMSFT

Copy link
Copy Markdown
Member Author

/ba-g Known failure - #113652

@AaronRobinsonMSFT
AaronRobinsonMSFT merged commit a3d6c25 into dotnet:main Mar 19, 2025
@AaronRobinsonMSFT
AaronRobinsonMSFT deleted the hmf_strcns branch March 19, 2025 21:19
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants