NilFoundation / zkLLVM

Zero-Knowledge Proof Systems Circuit Compiler
https://docs.nil.foundation/zkllvm
273 stars 44 forks source link

Move asserts definition to crypto3 #583

Open makxenov opened 3 months ago

makxenov commented 3 months ago

Asserts from assigner could be used in another repositories as well, so we decided to move it to crypto3. Required steps:

aleasims commented 3 months ago

It would be nice also to have this variant of todo function as well:

void todo(const std::string &msg, const char *filename, unsigned line);

Right now you can't write:

std::string s("world");
TODO("hello " + s);

I missed this variant when I was implementing this.

aleasims commented 3 months ago

We have some custom asserts defined in zkllvm-blueprint as well (here). Maybe we can replace them also. Blueprint is already using crypto3 library anyways.