Make nearest_mult8 concise
vitkarpov opened this issue · 5 comments
I'd like to work on this just to dip my toes into the project.
Here is the current implementation of nearest_mult8 function →
Line 14 in 9a65fe6
I'd like to shorten it to sz = (sz + 7) & ~7
and build & run tests along the way.
Oh wow! that seems pretty cool and concise! I'd like to note that tests are still a WIP so please don't rely on them just yet. Most of them may not even work right now (probably due to lack of functions).
But, you can test the memory using any simple test (say tests/facto_iterate.fer
) and prepending the cmake command with MEM_PROFILE=true
(such that the command becomes MEM_PROFILE=true cmake ..
)
That will show the total allocated bytes
which should hold same values for both before and after modification. 😄
There is still a lot to be done in the project - from writing test cases to CI, as well as library functions 😅, so I am working on it.
Thanks for helping out ❤️
Great! Count this issue on me then 👌
Awesome! Assigned to you.
Thanks again! 😁