Re: scratch/comp-static-data 5aa3db2f11: comp: Add support for compiling elisp constants into static data.

[Andrea Corallo]

Vibhav Pant <vibhavp@gmail.com> writes:

[...]

> Hi Andrea,
>
> This branch was mostly inspired by your comment in `emit_static_object'
> that mentioned how libgccjit didn't support static initializers
> (presumably back then). Now that there is some support for initializing
> static globals with constant expressions, I have been working on this
> patch for the last few weeks with the aim of trying to reduce the use
> of `read' for initializing constants referenced by elisp code
> (therefore drastically reducing the amount of heap allocations and
> improving eln load times), by compiling them into the eln as static
> (optionally const) data. Think of it as every eln module having its own
> pure storage.
>
> There are still a bunch of TODOs, the first one being reducing the size
> of the generated binaries (which is mostly due to how `comp_init_obj'
> works right now). Nonetheless, I decided to make this code public once
> it was passing basic sanity tests (including the comp-tests.el suite)
> and was in a more or less working condition as a daily driver. As a
> result, the branch is still a WIP, but I will try to document more of
> the code moving forward.
>
> Best,
> Vibhav

Hi Vibhav,

as usual when you are ready I suggest you to show the pseudo C generated
with native-comp-debug >= 2 for the stock system and for the one with
your patch.  This for a _minimal_ example.

Thanks for working on this.

  Andrea