Skip to content

[simd] Resolve LWG4420 and LWG4393 #8353

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[simd] Resolve LWG4420 and LWG4393 #8353

wants to merge 1 commit into from

Conversation

@mattkretz
Copy link
Member

@mattkretz mattkretz commented Oct 21, 2025

This provides the proposed resolution to LWG4420, which then also resolves LWG4393.

@jensmaurer jensmaurer added lwg Issue must be reviewed by LWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. labels Oct 21, 2025
@jensmaurer
Copy link
Member

jensmaurer commented Oct 21, 2025 •
edited
Loading

This pull request has nothing to do with the current proposed resolution on the mentioned LWG issues.
Also, where did the "output_range" constraint go that would address LWG4393?
And please use the correct git commit title per https://github.com/cplusplus/draft/wiki/Commit-message-format .
And we probably have an overfull \hbox or so; the CI build is failing.

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Oct 21, 2025
@mattkretz
Copy link
Member Author

mattkretz commented Oct 22, 2025

Sorry @jensmaurer, I did not expect a review as long as the PR is in draft status.
I just sent a big update to LWG4420 and wanted to point to a PR from it, which is why I opened it here. I'll fix the hboxes and commit message.

@mattkretz mattkretz force-pushed the LWG4420 branch 2 times, most recently from 1b1e99f to c081cf4 Compare October 24, 2025 18:58
@mattkretz
LWG4420 [simd] conversions (constructor, load, stores, gather, and sc…
cc244eb 
…atter) are incorrectly constrained for <stdfloat> types

Also fixes LWG4393, NB DE-288 (C++26 CD), NB DE-285 (C++26 CD).
@mattkretz
Copy link
Member Author

mattkretz commented Nov 10, 2025

this is outdated and would need a new PR from my new fork anyway

@mattkretz mattkretz closed this Nov 10, 2025
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 10, 2025

All the forks have been restored -- could you have another look?

@mattkretz
Copy link
Member Author

mattkretz commented Nov 11, 2025

Yes, I can update it with the last changes. How does this work wrt. #8461? (It's one part of LWG Motion 2)

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

Reviewers

No reviews

Assignees

No one assigned

Labels

changes requested Changes to the wording or approach have been requested and not yet applied. lwg Issue must be reviewed by LWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mattkretz @jensmaurer @tkoeppe