privacy-scaling-explorations / chiquito

DSL for Halo2 circuits
https://docs.pecadorplonkish.xyz/
185 stars 39 forks source link

Add missing arithmetic operators to parser #216

Closed leolara closed 6 months ago

leolara commented 7 months ago

We should have all that are in section "Arithmetic operators" and "Bitwise operators" here https://docs.circom.io/circom-language/basic-operators/#arithmetic-operators

The operator priority should be like in circom that is the same as in rust.

rutefig commented 6 months ago

In this branch: https://github.com/privacy-scaling-explorations/chiquito/tree/rutefig/205-add-missing-binary-operators-to-lalrpop-and-ast It is linked to the issue #205

leolara commented 6 months ago

@rutefig why do you link to a branch instead of sending a PR?

rutefig commented 6 months ago

@rutefig why do you link to a branch instead of sending a PR?

It was just to keep track that this issue is being done on the same branch as the issue #205 because it is related, when the PR is ready I will link it here too