On Fri, 11 Jan 2019 13:00:22 -0700
Jeff Law <l...@redhat.com> wrote:
> On 12/30/18 4:51 AM, Jozef Lawrynowicz wrote:
> >
> > The attached patch adds an assertion that SET_{DECL,TYPE}_ALIGN is called
> > with
> > a value which is a power of 2, or 0, for targets which support a partial int
> > mode. This should catch issues in the future with non-power of 2
> > alignments being set, which could propagate into serious problems later in
> > the
> > compilation process.
> >
> Let's defer to gcc-10.
>
> I'm not sure it's worth trying to conditionalize this on the partial
> integer modes. Why not just do it all the time? I'd be surprised if
> it's that expensive.
Yes in hindsight it was overkill to conditionalize the checks.
I'll redo and submit this again with yours and Richard's suggestions for
GCC-10.
Thanks,
Jozef
