-
Notifications
You must be signed in to change notification settings - Fork 285
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
Stack checks via pointer to over-aligned stack allocation #781
Labels
optimization
Iproves performance without functional changes
Comments
chfast
added a commit
that referenced
this issue
May 28, 2024
Use `std::aligned_alloc()` (and `_aligned_malloc()` on MSVC) to allocate the memory for EVM stack. This removes unnecessary initialization of the memory to zero (closes #403). The memory alignment is set to 32 bytes (as previously) but allows transitioning to "over-aligned" allocation for additional optimization (see #781).
chfast
added a commit
that referenced
this issue
May 28, 2024
Use `std::aligned_alloc()` (and `_aligned_malloc()` on MSVC) to allocate the memory for EVM stack. This removes unnecessary initialization of the memory to zero (closes #403). The memory alignment is set to 32 bytes (as previously) but allows transitioning to "over-aligned" allocation for additional optimization (see #781).
chfast
added a commit
that referenced
this issue
May 28, 2024
Use `std::aligned_alloc()` (and `_aligned_malloc()` on MSVC) to allocate the memory for EVM stack. This removes unnecessary initialization of the memory to zero (closes #403). The memory alignment is set to 32 bytes (as previously) but allows transitioning to "over-aligned" allocation for additional optimization (see #781).
chfast
added a commit
that referenced
this issue
May 28, 2024
Use `std::aligned_alloc()` (and `_aligned_malloc()` on MSVC) to allocate the memory for EVM stack. This removes unnecessary initialization of the memory to zero (closes #403). The memory alignment is set to 32 bytes (as previously) but allows transitioning to "over-aligned" allocation for additional optimization (see #781).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: