-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: implement modulesToCount selection from module limits TOML config #1443
base: arith-dev
Are you sure you want to change the base?
Conversation
# Conflicts: # arithmetization/src/main/java/net/consensys/linea/zktracer/module/hub/Hub.java
standardModulesToTrace = initStandardModulesToTrace(); | ||
modulesToCountMap = initModulesToCountMap(); | ||
|
||
modules = Stream.concat(standardModules.stream(), precompileLimitModules.stream()).toList(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is modules the modules triggered during execution of the evm? If yes, we don't need the precompileLimitModule in, they are now manually triggered by other modules when needed
l2L1Logs); | ||
} | ||
|
||
private List<Module> initStandardModulesToTrace() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe, the standardModuleToTrace (and to Count) could be initStandardModule + some few others (just take care if some order are needed, like for mum/mmio at tracing time)
@@ -905,7 +962,7 @@ public final MessageFrame messageFrame() { | |||
private void handleStack(MessageFrame frame) { | |||
this.currentFrame() | |||
.stack() | |||
.processInstruction(this, frame, MULTIPLIER___STACK_HEIGHT * (stamp() + 1)); | |||
.processInstruction(this, frame, MULTIPLIER___STACK_HEIGHT * stamp()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
#1395
Copied over from Tsvetan's PR#1396
This PR enable the dynamic inclusion/exclusion of modules for the line count RPC response: