This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add testcase for PR59250
- From: Marek Polacek <polacek at redhat dot com>
- To: Richard Biener <richard dot guenther at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 26 Nov 2013 10:24:06 +0100
- Subject: Re: [PATCH] Add testcase for PR59250
- Authentication-results: sourceware.org; auth=none
- References: <20131125094457 dot GL30062 at redhat dot com> <20131125094956 dot GV892 at tucnak dot redhat dot com> <20131125103637 dot GN30062 at redhat dot com> <20131125104729 dot GW892 at tucnak dot redhat dot com> <20131125110244 dot GO30062 at redhat dot com> <CAFiYyc0aJux8L0vGjw8-J_qAiZVRV64czNGkQHegPx=nX1MKtg at mail dot gmail dot com>
On Tue, Nov 26, 2013 at 10:13:35AM +0100, Richard Biener wrote:
> On Mon, Nov 25, 2013 at 12:02 PM, Marek Polacek <polacek@redhat.com> wrote:
> > On Mon, Nov 25, 2013 at 11:47:29AM +0100, Jakub Jelinek wrote:
> >> That looks much better, I wonder if it would reproduce even if the result
> >> is saved somewhere (either E *e; field in S with swapping of the two class
> >> definitions, or global var or static data member static E *e; of S),
> >> otherwise the new just allocates and throws away. Ok with that change, or
> >> if it makes the problem no longer reproduceable, ok as is.
> >
> > It is reproduceable with all three variants. I'll commit this one shortly.
> >
> >> Note, once the LTO issues with internal functions are fixed, we need
> >> to grep for all those dg-skip-if and remove them.
> >
> > Yeah, definitely. I'm keeping it in mind.
>
> I'll pre-approve removing the assert for ->leaf that currently makes
> internal fns ICE with LTO for now. If it makes them "work" (please
> double-check also code generation results).
Unfortunately, only removing the assert for ->leaf does not work,
without it I get:
./a.out
[Leaving LTRANS /tmp/ccvr1cAT.args]
[Leaving LTRANS /tmp/ccfdPwKs.ltrans.out]
[Leaving LTRANS /tmp/cctcbfzL.args]
[Leaving LTRANS /tmp/ccfdPwKs.ltrans0.o]
/tmp/ccfdPwKs.ltrans0.ltrans.o: In function `main':
ccfdPwKs.ltrans0.o:(.text+0x1d): undefined reference to `.Lubsan_data0.2659'
(Though it may be a bug in ubsan.)
Marek