Enable support of u16
#4908
Assignees
Labels
enhancement
New feature or request
Comments
github-merge-queue bot
pushed a commit
that referenced
this issue
May 7, 2024
# Description ## Problem\* Resolves #4908 ## Summary\* Adds cases for `u16`. `i16` was right there so added at the same time. ## Additional Context ## Documentation\* Check one: - [ ] No documentation needed. - [x] Documentation included in this PR. - [ ] **[For Experimental Features]** Documentation to be submitted in a separate PR. # PR Checklist\* - [x] I have tested the changes locally. - [x] I have formatted the changes with [Prettier](https://prettier.io/) and/or `cargo fmt` on default settings. --------- Co-authored-by: Tom French <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
When #4162 was proposed and worked on, the AztecVM was uncertain about the support of
u16and the type was hence temporarily disabled.Happy Case
With the uncertainty now resolved and support for
u16built in AztecVM, we should re-enable support ofu16for more consistent integer data types and better development flexibility.Project Impact
None
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
Credit to @smanilov for flagging the gap.
#4162 (comment)
Would you like to submit a PR for this Issue?
None
Support Needs
No response
The text was updated successfully, but these errors were encountered: