Skip to content

Conversation

tannergooding
Copy link
Member

This resolves dotnet/runtime#120508

@tannergooding tannergooding requested a review from a team as a code owner October 15, 2025 14:20
@Copilot Copilot AI review requested due to automatic review settings October 15, 2025 14:20
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds and enriches missing XML documentation for System.Runtime.Intrinsics vector types (Vector64, Vector128, Vector256, Vector512) to address issue #120508.

  • Replaces many placeholder "To be added." entries with meaningful summaries, type parameter, parameter, and returns descriptions.
  • Adds references for constant properties and removes some inheritdoc tags where explicit docs are now provided.
  • Introduces new remarks and exception tags, though a few contain grammatical and structural issues.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 13 comments.

File Description
xml/System.Runtime.Intrinsics/Vector64.xml Adds inheritdoc references for constant getters; enhances classification method docs.
xml/System.Runtime.Intrinsics/Vector128.xml Substantial expansion of summaries, parameters, returns, and exceptions; removes some inheritdoc tags.
xml/System.Runtime.Intrinsics/Vector256.xml Mirrors Vector64 constant inheritdoc additions and removes some inheritdoc tags.
xml/System.Runtime.Intrinsics/Vector512.xml Similar updates to Vector256 with inheritdoc additions and cleanup.

@tannergooding
Copy link
Member Author

-- GitHub seems to not like applying bulk suggestions for files with "large diffs". Loading the file causes it to say "the file has changed recently" and forces it to refresh, which then causes the changes to no longer be displayed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

System.Runtime.Intrinsics docs for .NET 10.0 APIs

2 participants