Skip to content

Conversation

eisenwave
Copy link
Member

Fixes NB US 175-281 (C++26 CD).

Fixes cplusplus/nbballot#856.

@eisenwave eisenwave added the ballot-comment Response to an NB or ISO comment on a ballot label Oct 9, 2025
@eisenwave
Copy link
Member Author

I also wrote down some thoughts on the issue at cplusplus/nbballot#856 (comment)

We could also reorder the subclauses, but in my opinion, they're structured very logically. All the basic_vec stuff first, then basic_mask. I don't see any compelling reason to deviate, and making the synopsis match the subclauses is the easiest fix.

@mattkretz can you confirm that the mismatch between the synopsis and the subclauses is accidental?

@mattkretz
Copy link
Member

@mattkretz can you confirm that the mismatch between the synopsis and the subclauses is accidental?

The current order is derived from what got published in the TS. And the TS order was probably fairly accidental. Making the synopsis match the subclause ordering (rather than the other way around) seems like the safest thing to do.

FWIW, I implement basic_mask before basic_vec because that's just more natural in terms of code dependencies. The order in my implementation (not asking for a subclause reordering):

  1. [simd.expos]
  2. [simd.traits]
  3. [simd.iterator]
  4. [simd.creation] + [simd.permute.static]
  5. [simd.mask] including all hidden friends in [simd.mask.nonmembers]
  6. [simd.flags]
  7. [simd.class] including all hidden friends in [simd.nonmembers]
  8. [simd.loadstore]
  9. remaining [simd.permute.*]
  10. [simd.mask.reductions]
  11. [simd.reductions]
  12. [simd.alg]
  13. [simd.bit]
  14. complex specializations of [simd.mask] and [simd.class]
  15. [simd.math]
Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tkoeppe , we need this to go in so that, as a next step, we can adjust the subclause headings quoted in the synopsis to the new reality, without causing merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ballot-comment Response to an NB or ISO comment on a ballot

4 participants