Re: [PATCH v4 0/4] Implement extents from the mdspan header.

2025-04-18 Thread Tomasz Kaminski
The whole patch series now LGTM to me. Note, that you need separate approval from the maintainer (most likely from Jonathan) for merging. On Fri, Apr 18, 2025 at 1:32 PM Luc Grosheintz wrote: > This is the fourth interation and replaces: > https://gcc.gnu.org/pipermail/libstdc++/2025-April/06104

Re: [PATCH v4 0/4] Implement extents from the mdspan header.

2025-04-18 Thread Tomasz Kaminski
On Fri, Apr 18, 2025 at 1:56 PM Luc Grosheintz wrote: > I have a question about how to proceed. First we finish the > refinement of the current commits. Then I could: > > 1. start a new patch series for layout_*; > 2. or add new commits to the current series for layout_*. > My personal preference

Re: [PATCH v4 0/4] Implement extents from the mdspan header.

2025-04-18 Thread Luc Grosheintz
I have a question about how to proceed. First we finish the refinement of the current commits. Then I could: 1. start a new patch series for layout_*; 2. or add new commits to the current series for layout_*. The advantage of 2. is that we can do more tests before merging anything into trunk. Fo

[PATCH v4 0/4] Implement extents from the mdspan header.

2025-04-18 Thread Luc Grosheintz
This is the fourth interation and replaces: https://gcc.gnu.org/pipermail/libstdc++/2025-April/061046.html Changes since v3: * Use `_S_` prefix for static member functions. * Use consteval to reduce code size. * Simplify ctors of _ExtentsStorage. * Added a test to cover the mandates of std