-
Notifications
You must be signed in to change notification settings - Fork 169
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
Performance degradation on M1 when moving from Rust 1.69 to 1.70 #917
Comments
Seems like the issue is with BLAKE3 hash function. For w/e reason it became more than 50% slower. |
Very interesting! Have you by chance compared using RPO? |
Good question! It actually seems a similar thing happened with RPO - so, maybe something else is going on here. Compiled with Rust 1.69:
Compiled with Rust 1.70:
GPU-acceleratedWhen running in GPU-accelerated mode, the differences are barely noticeable: Compiled with Rust 1.69:
Compiled with Rust 1.70:
|
It seems like this issue has resolved itself with the current Rust nightly (1.74). And in general, the proving to quite a bit faster due to latest trace optimizations:
Assuming the performance stays the same, we should set the minimum supported Rust version to 1.74 once it becomes stable. |
Closed by #1084 since we now set min required |
Seems like when compiled with Rust 1.70, Miden VM runs about 13% slower than when compiled with Rust 1.69. I've included the benchmarks for running the VM for$2^{20}$ cycles below. Interestingly, it seems like the thing that really slowed down is hashing (which itself could be caused by a number of things), while most other steps have actually improved in performance.
Compiled with Rust 1.69:
Compiled with Rust 1.70:
The text was updated successfully, but these errors were encountered: