Feral-Lang/Feral

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 →

size_t nearest_mult8( size_t sz )

I'd like to shorten it to sz = (sz + 7) & ~7 and build & run tests along the way.

@Electrux hey 👋 Let me know what you think, thanks!

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! 😁