Skip to content

Fix InstructionSetDesc.txt to only put managed names for ISAs with managed names#123148

Merged
jkoritzinsky merged 1 commit into
dotnet:mainfrom
jkoritzinsky:isa-public-types-fix
Jan 14, 2026
Merged

Fix InstructionSetDesc.txt to only put managed names for ISAs with managed names#123148
jkoritzinsky merged 1 commit into
dotnet:mainfrom
jkoritzinsky:isa-public-types-fix

Conversation

@jkoritzinsky

Copy link
Copy Markdown
Member

Now that we have logic that maps from InstructionSet enums to managed types, makes sure that the table is actually correct (managed names only for managed types, R2R names for ones without managed types).

Fixes #123139

…naged names

Now that we have logic that maps from InstructionSet enums to managed types, makes sure that the table is actually correct (managed names only for managed types, R2R names for ones without managed types).
@github-actions github-actions Bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 13, 2026

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jkoritzinsky jkoritzinsky added area-ReadyToRun and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jan 13, 2026
@jkoritzinsky jkoritzinsky merged commit 4addbfa into dotnet:main Jan 14, 2026
103 of 106 checks passed
@jkoritzinsky jkoritzinsky deleted the isa-public-types-fix branch January 14, 2026 01:15
@github-project-automation github-project-automation Bot moved this to Done in AppModel Jan 14, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

3 participants