C bindings for verkle ipa #227
Conversation
Signed-off-by: Nischal Sharma <[email protected]>
| } | ||
|
|
||
| // Declare native methods | ||
| public static native int ipa_commit(byte[] r, int r_len, byte[] inputs, int inputs_len); |
There was a problem hiding this comment.
You might need to change the params of these functions according to current implementation. For example like this -
There was a problem hiding this comment.
yes for verkle we need all the methods listed in LibIpaMulitpoint
you can ignore verifyPreStateRoot for the moment
| // Declare native methods | ||
| public static native int ipa_commit(byte[] r, int r_len, byte[] inputs, int inputs_len); | ||
| public static native int ipa_prove(byte[] r, int r_len, byte[] inputs, int inputs_len); | ||
| public static native int ipa_verify(byte[] r, int r_len, byte[] inputs, int inputs_len); |
There was a problem hiding this comment.
Same for all other functions, check with current implementation of what args it is taking and what should be the return type.
| public void testCommit() { | ||
| byte[] inputs = new byte[128]; | ||
|
|
||
| byte[] result = LibConstantineEIP196.add(inputs); |
| } | ||
|
|
||
| // Verkle functions | ||
| int ipa_commit_aff(const ctt_eth_verkle_ipa_polynomial_eval_crs *crs, banderwagon_ec_aff res, const ctt_eth_verkle_ipa_polynomial_eval_poly *poly) { |
There was a problem hiding this comment.
Update JAVA bindings functions according to these
|
Also sign DCO for all commits |
|
you have a list of tests in this package that you can use to verify your implementation https://github.com/hyperledger/besu-native/tree/d5504630332af07a22d519436dcfcbda2de3b134/ipa-multipoint/src/test/java/org/hyperledger/besu/nativelib/ipa_multipoint . |
No description provided.