[PATCH v4 1/2] libstdc++: Refactor _ScalarAbi<N> into _Abi<N, N>

Matthias Kretz MatthiasKretz@gmx.net
Wed Jul 1 11:40:53 GMT 2026


Jonathan Wakely [Wednesday, 1 July 2026, 12:42:41 CEST]:
> >-  /** @internal
> >-   * This ABI tag describes basic_vec objects that store one or more
> >objects declared with the -   * [[gnu::vector_size(N)]] attribute.
> >-   * Applied to basic_mask objects, this ABI tag either describes
> >corresponding vector-mask objects -   * or bit-mask objects. Which one is
> >used is determined via @p _Var. +   * @p _Nreg determines the number of
> >recursive basic_vec/basic_mask data members where @p _Nreg is +   * equal
> >to 1. With @p _Nreg equal to 1, the basic_vec/basic_mask holds one vector
> >builtin (@p +   * _Np greater than 1) or a scalar (@ _Np equal to 1).
> 
> Does this work correctly with Doxygen, having a line break between the
> @p and the _Np name? There's certainly a 'p' missing in the last @p
> here.

Good question. Right now I'm flying blind on Doxygen. All bits/simd_* and 
bits/vec_ops.h are currently excluded. 
1. because I don't have a @file doxygen comment in any of them
2. INTERNAL_DOCS is set to NO

OK, so I just checked how _Abi's HTML turns out... The line break between @p 
and _Np is no problem.

> You could drop the @p and use `_Np` instead.

> The patch is OK for trunk with the missing 'p' fixed, and preferably
> just switching to markdown.

OK, I'll change the new comments blocks in this patch to use markdown then.

-- 
───────────────────────────────────────────────────────────────────
 Matthias Kretz                        https://www.kretzfamily.de/
───────────────────────────────────────────────────────────────────
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: This is a digitally signed message part.
URL: <https://gcc.gnu.org/pipermail/libstdc++/attachments/20260701/7f39b233/attachment.sig>


More information about the Libstdc++ mailing list