zkevm-circuits icon indicating copy to clipboard operation
zkevm-circuits copied to clipboard

Unify tables.rs table methods into `LookupTable` trait fns.

Open CPerezz opened this issue 2 years ago • 1 comments

Almost all of the lookup tables defined in tables.rs actually have the methods construct, assignments, load...

I think we could integrate all these inside of the LookupTable trait. Specially after the first refactor of this trait. Which was done in #1137

CPerezz avatar Feb 06 '23 16:02 CPerezz

The word lo/hi refactor touches most of the Tables tables.rs. I suggest we block this issue until https://github.com/privacy-scaling-explorations/zkevm-circuits/issues/1221 is completed.

ed255 avatar Jun 02 '23 09:06 ed255