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.

Sign up for GitHub

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

Jump to bottom

Compress memory allocations for repeated parameters #36

Merged

keynmol merged 1 commit into main from compress-allocations

Jan 22, 2022

Contributor

keynmol commented Jan 22, 2022

Closes #27

Reduce the number of calls to alloc when parameter types are repeated.
Add tests that instrument local Zone allocations


          Compress memory allocations for repeated parameters

3865d2b

keynmol commented

View reviewed changes

bindgen/src/main/scala/render/function.scala

+                        if returnAsWell then
+                          val block = s"__ptr_${typeIndices(f.returnType)}"
+                          val locationInBlock = usedAllocations(f.returnType)
+                          params ++ Map(-1 -> s"($block + $locationInBlock)")

Contributor Author

keynmol Jan 22, 2022

I am not proud of this. But it will do for now.

keynmol merged commit e43b17e into main

keynmol deleted the compress-allocations branch

January 22, 2022 18:32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet