Skip to content

chore: remove bigfield pow with witness exponent.#14563

Merged
suyash67 merged 1 commit into
nextfrom
sb/remove-bigfield-pow
May 31, 2025
Merged

chore: remove bigfield pow with witness exponent.#14563
suyash67 merged 1 commit into
nextfrom
sb/remove-bigfield-pow

Conversation

@suyash67

Copy link
Copy Markdown
Contributor

[WIP] Removes the pow(field_t exponent) from the bigfield class as it is not needed anywhere, and it is safer to remove it rather than using an unsafe implementation.

@suyash67 suyash67 changed the title remove bigfield pow with witness exponent. chore: remove bigfield pow with witness exponent. May 27, 2025
@suyash67 suyash67 changed the title chore: remove bigfield pow with witness exponent. chore: remove bigfield pow with witness exponent. May 27, 2025

@Rumata888 Rumata888 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@suyash67 suyash67 force-pushed the sb/remove-bigfield-pow branch from ced9d63 to a5ee499 Compare May 30, 2025 13:53
@suyash67 suyash67 added this pull request to the merge queue May 31, 2025
Merged via the queue into next with commit 32612f8 May 31, 2025
5 of 6 checks passed
@suyash67 suyash67 deleted the sb/remove-bigfield-pow branch May 31, 2025 22:22
github-merge-queue Bot pushed a commit that referenced this pull request Jun 17, 2025
Made sure that the current impl of `field_t` `pow()` matches the audited
(and
[deprecated](#14563))
implementation of `bigfield` `pow()`.

The `pow` method taking const integers as an argument is taken from
#14610
danielntmd pushed a commit to danielntmd/aztec-packages that referenced this pull request Jul 16, 2025
Made sure that the current impl of `field_t` `pow()` matches the audited
(and
[deprecated](AztecProtocol#14563))
implementation of `bigfield` `pow()`.

The `pow` method taking const integers as an argument is taken from
AztecProtocol#14610
AztecBot pushed a commit to AztecProtocol/barretenberg that referenced this pull request Dec 3, 2025
Made sure that the current impl of `field_t` `pow()` matches the audited
(and
[deprecated](AztecProtocol/aztec-packages#14563))
implementation of `bigfield` `pow()`.

The `pow` method taking const integers as an argument is taken from
AztecProtocol/aztec-packages#14610
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants