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/
───────────────────────────────────────────────────────────────────

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to