[PATCH] Add gcc_jit_type_get_aligned
David Malcolm
dmalcolm@redhat.com
Sun Jan 1 00:00:00 GMT 2017
On Fri, 2017-03-31 at 17:13 -0400, David Malcolm wrote:
> On Thu, 2017-03-30 at 22:28 +0200, Florian Weimer wrote:
> > * David Malcolm:
> >
> > > Here's a work-in-progress implementation of the idea, adding this
> > > entrypoint to the API:
> > >
> > > extern gcc_jit_type *
> > > gcc_jit_type_get_aligned (gcc_jit_type *type,
> > > unsigned int alignment_in_bytes);
> >
> > Should be size_t, not unsigned int. A 2**31 alignment isn't as
> > ridiculous as it might seem. x86-64 already has a 2**30 alignment
> > requirement in some contexts.
>
> Thanks; fixed in this version.
>
> Here's a completed version of the patch.
>
> It also implements the missing C++ binding
> gccjit::type::get_const, needed by a test case.
>
> Successfully bootstrapped®rtested on x86_64-pc-linux-gnu.
> Takes jit.sum from 8609 to 9349 PASS results.
>
> Release managers: is it acceptable to commit this to trunk in
> stage 4? It purely touches jit-related code/testcases, but I
> appreciate it's very late to be adding features.
>
> Otherwise I'll commit it in the next stage 1.
I've committed this to trunk (for gcc 8) as r247111.
[...snip...]
More information about the Jit
mailing list