On Mon, Oct 27, 2025 at 11:36 AM Matthias Kretz <[email protected]> wrote:
> Matthias Kretz [Friday, 24 October 2025, 18:09:42 CET]:
> > Tomasz Kaminski [Friday, 24 October 2025, 17:40:40 CEST]:
> > > > #endif // __glibcxx_integer_sequence
> > > >
> > > > +#if __cpp_structured_bindings >= 202411L
> > > > +#if __has_builtin(__integer_pack)
> > > > + template <auto _Num, typename _Tp = decltype(_Num)>
> > > > + inline constexpr _Tp
> > > > + _IotaArray[_Num] = {__integer_pack(_Tp(_Num))...};
> > > > +#else
> > >
> > > This should be #elif __glibcxx_integer_sequence, we dependent on that.,
> > > Otherwise this looks good to me.
> >
> > But then, shouldn't I rather move the `#endif //
> __glibcxx_integer_sequence`
> > from above below my code?
> >
> > So rather guard it with the following?
> >
> > __cpp_structured_bindings >= 202411L || __cpp_expansion_statements >=
> > 202506L
>
> I'll commit with only the change you requested then. (I want my next
> [simd]
> patch to depend on this. So I'd like to get this out of the way.)
>
I do not have approval rights, so before any patch gets merged, it needs
to be approved by Jonathan.
>
> - Matthias
>
> --
> ──────────────────────────────────────────────────────────────────────────
> Dr. Matthias Kretz https://mattkretz.github.io
> GSI Helmholtz Center for Heavy Ion Research https://gsi.de
> std::simd
> ──────────────────────────────────────────────────────────────────────────
>
