Skip to content
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

Let memory sizes of the various IMEMO object types be reflected correctly #2140

Closed
wants to merge 1 commit into from

Conversation

methodmissing
Copy link
Contributor

@methodmissing methodmissing commented Apr 22, 2019

Supersedes #2112. In current trunk only the imemo_tmpbuf type's auxiliary malloc heap buffer is factored into obj_memsize_of. The following IMEMO types also allocate on the malloc heap:

  • imemo_env (similar pattern as imemo_tmpbuf, an array of VALUEs)
  • imemo_ment (single struct allocated)
  • imemo_iseq (massaged the iseq_memsize API a little)
  • imemo_ast (more difficult to get right, need to walk buffer elements too)

The imemo_memsize function introduced attempts to be the entry point for object size calculation of the IMEMO types.

@ko1
Copy link
Contributor

ko1 commented Jul 23, 2019

Just curious. How to use them?

@hsbt hsbt added the Backport label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 participants