[PATCH][AArch64] Use intrinsics for widening multiplies (PR91598)

Andrew Pinski pinskia@gmail.com
Mon Mar 9 17:30:53 GMT 2020


On Mon, Mar 9, 2020 at 10:26 AM Wilco Dijkstra <Wilco.Dijkstra@arm.com> wrote:
>
> Hi Christophe,
>
> > I noticed a regression introduced by Delia's patch "aarch64: ACLE
> > intrinsics for BFCVTN, BFCVTN2 and BFCVT":
> > (on aarch64-linux-gnu)
> > FAIL: g++.dg/cpp0x/variadic-sizeof4.C  -std=c++14 (internal compiler error)
> >
> > I couldn't reproduce it with current ToT, until I realized that your
> > patch fixes it. However, I'm wondering whether that's expected given
> > the context of both patches....
>
> It sounds like this is memory corruption. Neither patch should have changed
> anything in the C++ frontend.

It sounds like some GC issue.  The patch would have changed a few
things related to the front-end though.  Mainly the decl UIDs do
increase due to the new builtins.  Note most likely Deli's patch did
the same too.

Thanks,
Andrew Pinski

>
> Cheers,
> Wilco
>


More information about the Gcc-patches mailing list