Skip to content

<mdspan>: Use concepts everywhere and add mdspan deduction guides - #3564

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:feature/mdspan2from
JMazurkiewicz:mdspan/all-concepts
Mar 14, 2023
Merged

<mdspan>: Use concepts everywhere and add mdspan deduction guides#3564
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:feature/mdspan2from
JMazurkiewicz:mdspan/all-concepts

Conversation

@JMazurkiewicz

Copy link
Copy Markdown
Contributor
  • Replace (almost) all uses of enable_if with concepts,
  • Add missing mdspan deduction guides.

Drive-by:

  • Remove unnecessary includes,
  • Remove 2 non-standard constructors (one from layout_stride::mapping and one from mdspan),
  • Improve some comments,

Comment thread stl/inc/mdspan
Comment thread stl/inc/mdspan
Comment thread stl/inc/mdspan
@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 187a036 into microsoft:feature/mdspan2 Mar 14, 2023
@JMazurkiewicz
Jakub Mazurkiewicz (JMazurkiewicz) deleted the mdspan/all-concepts branch March 14, 2023 00:36
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for the high-concept PR! 😻

Jakub Mazurkiewicz (JMazurkiewicz) added a commit to JMazurkiewicz/STL that referenced this pull request Mar 17, 2023
Jakub Mazurkiewicz (JMazurkiewicz) added a commit to JMazurkiewicz/STL that referenced this pull request Mar 25, 2023
Jakub Mazurkiewicz (JMazurkiewicz) added a commit to JMazurkiewicz/STL that referenced this pull request Mar 25, 2023
Jakub Mazurkiewicz (JMazurkiewicz) added a commit to JMazurkiewicz/STL that referenced this pull request Mar 25, 2023
Jakub Mazurkiewicz (JMazurkiewicz) added a commit to JMazurkiewicz/STL that referenced this pull request Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mdspan C++23 mdspan

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants