Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Implement Directive::ToLeRadix in Brillig #473

Closed
Tracked by #395
TomAFrench opened this issue Aug 3, 2023 · 1 comment
Closed
Tracked by #395

Implement Directive::ToLeRadix in Brillig #473

TomAFrench opened this issue Aug 3, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@TomAFrench
Copy link
Member

Problem

We're wanting to remove directives but the toLeRadix has no Brillig implementation which prevents us from doing so.

Happy Case

Directive::toLeRadix should be reimplemented in brillig so that we can remove this directive.

Alternatives Considered

Rather than having a handwritten Brillig implementation this could potentially be done as an unconstrained function in Noir.

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response
@TomAFrench TomAFrench added the enhancement New feature or request label Aug 3, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Aug 3, 2023
@TomAFrench TomAFrench mentioned this issue Aug 3, 2023
Closed
@TomAFrench
Copy link
Member Author

Closed in noir-lang/noir#2012

@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Noir Aug 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
Noir
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TomAFrench