Skip to content
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

feat: add BitVec.[toNat|toInt|toFin|getLsbD|getMsbD|getElem|msb]_fill #6177

Merged
merged 8 commits into from
Jan 8, 2025

Conversation

tobiasgrosser
Copy link
Contributor

@tobiasgrosser tobiasgrosser commented Nov 22, 2024

This PR implements BitVec.*_fill.

We also add toInt_allOnes and toFin_allOnes as the former is needed here. This completes the allOnes API.

@tobiasgrosser
Copy link
Contributor Author

changelog-library

@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Nov 22, 2024
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Nov 22, 2024

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 5145030ff4bed884213f584f8b2402be82e96d45 --onto 72e952eadc6a171310f1d8e9d6e78acf98421494. (2024-11-22 22:09:01)
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase f6e88e5a0525b744c5df018dd96d2f0cbca29435 --onto c5181569f959e4a0d9586954212125adcb6e44d0. (2024-12-05 06:04:23)

@kim-em
Copy link
Collaborator

kim-em commented Nov 23, 2024

An alternative approch here might be to deprecate/delete fill. Do you know why we have it in the first place?

@kim-em kim-em added the awaiting-author Waiting for PR author to address issues label Nov 26, 2024
@tobiasgrosser
Copy link
Contributor Author

An alternative approch here might be to deprecate/delete fill. Do you know why we have it in the first place?

It was added in 75272cb without further comments. Given that its a convenient helper and its easy to complete the API, I would be happy to keep it.

@tobiasgrosser
Copy link
Contributor Author

awaiting-review

@github-actions github-actions bot added awaiting-review Waiting for someone to review the PR and removed awaiting-author Waiting for PR author to address issues labels Dec 5, 2024
@leanprover-bot leanprover-bot added the P-high We will work on this issue label Dec 6, 2024
@kim-em kim-em removed the awaiting-review Waiting for someone to review the PR label Jan 8, 2025
@kim-em kim-em added this pull request to the merge queue Jan 8, 2025
Merged via the queue into leanprover:master with commit 9040108 Jan 8, 2025
18 checks passed
luisacicolini pushed a commit to opencompl/lean4 that referenced this pull request Jan 21, 2025
…leanprover#6177)

This PR implements `BitVec.*_fill`.

We also add `toInt_allOnes` and `toFin_allOnes` as the former is needed
here. This completes the allOnes API.
JovanGerb pushed a commit to JovanGerb/lean4 that referenced this pull request Jan 21, 2025
…leanprover#6177)

This PR implements `BitVec.*_fill`.

We also add `toInt_allOnes` and `toFin_allOnes` as the former is needed
here. This completes the allOnes API.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-library Library P-high We will work on this issue toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants