Implement to_be_radix builtin in the comptime interpreter
#6042
Labels
enhancement
New feature or request
Comments
5 tasks
|
Implemented here but this can be worked around by reversing in Noir for the time being now that it returns an array rather than a slice. |
github-merge-queue bot
pushed a commit
that referenced
this issue
Sep 16, 2024
# Description ## Problem\* Resolves #6042 ## Summary\* This PR implements `to_be_radix` in the comptime interpreter by reusing our impl for `to_le_radix`. ## Additional Context ## Documentation\* Check one: - [x] No documentation needed. - [ ] 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.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
That intrinsic is missing from the comptime interpreter
Happy Case
to_be_radixcan be used on aFieldtype in comptimeWorkaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
Blocker
Blocker Context
No response
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: