Skip to content

Commit

Permalink
update makefile targets
Browse files Browse the repository at this point in the history
This updates the existing Makefile targets `zkevm.bin` and
`zkevm_for_reference_tests.bin` to use `go-corset` instead of `corset`.
This is necessary because `corset` cannot compile the constraints any
longer as they contain features which the older tool does not support.

The alternative targets `zkevm.go.bin` and
`zkevm_for_reference_tests.go.bin` are now dropped, since they are no
longer required.
  • Loading branch information
DavePearce committed Jan 18, 2025
1 parent fadfc1c commit 35d6ca0
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -176,9 +176,6 @@ define.go: ${ZKEVM_MODULES}
${CORSET} wizard-iop -vv -o $@ ${ZKEVM_MODULES}

zkevm.bin: ${ZKEVM_MODULES}
${CORSET} compile -vv -o $@ ${ZKEVM_MODULES}

zkevm.go.bin: ${ZKEVM_MODULES}
${GO_CORSET} compile -o $@ ${ZKEVM_MODULES}


Expand Down Expand Up @@ -217,9 +214,6 @@ ZKEVM_MODULES_FOR_REFERENCE_TESTS := ${ALU} \
# ${HUB} \
zkevm_for_reference_tests.bin: ${ZKEVM_MODULES_FOR_REFERENCE_TESTS}
${CORSET} compile -vv -o $@ ${ZKEVM_MODULES_FOR_REFERENCE_TESTS}

zkevm_for_reference_tests.go.bin: ${ZKEVM_MODULES_FOR_REFERENCE_TESTS}
${GO_CORSET} compile -o $@ ${ZKEVM_MODULES_FOR_REFERENCE_TESTS}


Expand Down

0 comments on commit 35d6ca0

Please sign in to comment.