Skip to content

Lax conversions - #42

Merged
danieltowner merged 2 commits into
mainfrom
lax_conversions
Oct 21, 2026
Merged

danieltowner merged 2 commits into
mainfrom
lax_conversions

Conversation

@danieltowner

Copy link
Copy Markdown
Contributor

No description provided.

@rarutyun

rarutyun commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

I would suggest we are done with to_register PR (#41) because it's seems like there is a lot of overlap between those PRs.

gcc used lax vector conversion to provide implicit conversions between vectors of the same size but different types. This has been removed so that there is no requirement for gcc users to provide this flag themselves for their projects. A few minor issues that appeared as a consequence were fixed by inserting extra explicit conversions where needed.
Comment thread include/xvec/detail/mask.hpp Outdated
The implicit constructor could inadvertently be used for conversion from integer-like on compact-mask targets. Force the caller to explicitly use a named constructor called `from_builtin` to oprevent this happening.

@rarutyun rarutyun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks way better now

@danieltowner
danieltowner merged commit 49a8d00 into main Oct 21, 2026
4 checks passed
@danieltowner
danieltowner deleted the lax_conversions branch October 21, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants