Skip to content

Unused variable in __barrett_reduction #195

@shreyas-londhe

Description

@shreyas-londhe

Aim

Remove unused variable quotient_mul_modulus_normalized in __barrett_reduction.

Expected Behavior

The variable should be removed.

Bug

In the __barrett_reduction helper, the variable quotient_mul_modulus_normalized is said to be fitting in N limbs, which is wrong. Also the same variable is not used in the rest of the function and can be removed.
reference - https://github.com/noir-lang/noir-bignum/blob/main/src/fns/unconstrained_helpers.nr#L243-L246

To Reproduce

Workaround

None

Workaround Description

No response

Additional Context

No response

Project Impact

Nice-to-have

Blocker Context

No response

Nargo Version

No response

NoirJS Version

No response

Proving Backend Tooling & Version

No response

Would you like to submit a PR for this Issue?

None

Support Needs

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    Status

    👀 To Triage

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions