Skip to content

[Proposal] AllBitsSet for Scalar #678

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.

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

Closed
wants to merge 1 commit into from

Conversation

WhiteBlackGoose
Copy link
Contributor

@WhiteBlackGoose WhiteBlackGoose commented Nov 8, 2021

Similarly to .NET 5's Vector64.AllBitsSet (and other bitnesses) I suggest adding AllBitsSet public readonly field to Scalar<T>.

It is already implemented as internal field in #670 (so I propose making it public)

@WhiteBlackGoose WhiteBlackGoose changed the title [Proposal] AllBitsSet for Scalar and Simd [Proposal] AllBitsSet for Scalar Nov 8, 2021
@WhiteBlackGoose WhiteBlackGoose marked this pull request as ready for review November 8, 2021 10:59
@WhiteBlackGoose
Copy link
Contributor Author

Um, idk if it's a good idea to add it to an existing proposal...

@Perksey
Copy link
Member

Perksey commented Nov 8, 2021

I think this is small enough to just go in a PR directly. In the past we've modified historical proposals, but we all agreed in the 1.0 Preview era that this isn't really the purpose of them and just resulted in too much maintenance for the little things.

(you won't need to make a proposal for just one function)

From the (somewhat outdated) proposals README

@WhiteBlackGoose
Copy link
Contributor Author

I interpreted it as there's no need to create a new proposal document (so better to be added to old ones). 😄

I think this is small enough to just go in a PR directly

So I can just make it public, no other actions needed?

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