Skip to content

Implement P3029 and LWG-4351: Improving mdspan And span CTAD#5692

Open
EienMiku wants to merge 13 commits into
microsoft:mainfrom
EienMiku:span-P3029
Open

Implement P3029 and LWG-4351: Improving mdspan And span CTAD#5692
EienMiku wants to merge 13 commits into
microsoft:mainfrom
EienMiku:span-P3029

Conversation

@EienMiku

@EienMiku EienMiku commented Aug 24, 2025

Copy link
Copy Markdown

Implements P3029, makes span and mdspan' CTAD integral_constant-aware to deduce more efficient types.

Fixes #4523.

@EienMiku
EienMiku requested a review from a team as a code owner August 24, 2025 20:07
@github-project-automation github-project-automation Bot moved this to Initial Review in STL Code Reviews Aug 24, 2025
@EienMiku

Copy link
Copy Markdown
Author

@EienMiku please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

@StephanTLavavej

This comment was marked as resolved.

@StephanTLavavej StephanTLavavej added mdspan C++23 mdspan cxx26 C++26 feature labels Aug 25, 2025
@EienMiku

This comment was marked as resolved.

@frederick-vs-ja

This comment was marked as resolved.

@EienMiku

This comment was marked as resolved.

Comment thread stl/inc/span
@frederick-vs-ja

This comment was marked as resolved.

@EienMiku

This comment was marked as resolved.

@EienMiku

This comment was marked as resolved.

@YexuanXiao

This comment was marked as resolved.

@frederick-vs-ja

This comment was marked as resolved.

@AlexGuteniev

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

@EienMiku
EienMiku marked this pull request as draft July 14, 2026 14:01
Comment thread stl/inc/span Outdated
Co-authored-by: A. Jiang <de34@live.cn>

This comment was marked as resolved.

@EienMiku
EienMiku requested a review from frederick-vs-ja July 15, 2026 13:43
Comment thread tests/std/tests/P0009R18_mdspan_extents/test.cpp
Comment thread tests/std/tests/P0009R18_mdspan_mdspan/test.cpp
Comment thread tests/std/tests/P0122R7_span/test.cpp
Comment thread stl/inc/span
Copilot AI review requested due to automatic review settings July 15, 2026 16:34

This comment was marked as resolved.

@StephanTLavavej

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

@StephanTLavavej StephanTLavavej moved this from Initial Review to Work In Progress in STL Code Reviews Jul 16, 2026
@StephanTLavavej

This comment was marked as resolved.

@YexuanXiao

This comment has been minimized.

Comment thread tests/std/tests/P0009R18_mdspan_extents/test.cpp
Co-authored-by: A. Jiang <de34@live.cn>
Copilot AI review requested due to automatic review settings July 17, 2026 01:53

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

Comment thread stl/inc/span
@StephanTLavavej

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

Copilot AI review requested due to automatic review settings July 17, 2026 12:28

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

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

@StephanTLavavej

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

@StephanTLavavej StephanTLavavej moved this from Work In Progress to Initial Review in STL Code Reviews Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cxx26 C++26 feature mdspan C++23 mdspan

Projects

Status: Initial Review

Development

Successfully merging this pull request may close these issues.

P3029R1 Improving mdspan And span CTAD

7 participants