Re: [PATCH] Add testcase for PR59250

2013-11-26 Thread Marek Polacek
On Tue, Nov 26, 2013 at 10:13:35AM +0100, Richard Biener wrote: > On Mon, Nov 25, 2013 at 12:02 PM, Marek Polacek 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

Re: [PATCH] Add testcase for PR59250

2013-11-26 Thread Richard Biener
On Mon, Nov 25, 2013 at 12:02 PM, Marek Polacek 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 glob

Re: [PATCH] Add testcase for PR59250

2013-11-25 Thread Marek Polacek
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

Re: [PATCH] Add testcase for PR59250

2013-11-25 Thread Jakub Jelinek
On Mon, Nov 25, 2013 at 11:36:37AM +0100, Marek Polacek wrote: > On Mon, Nov 25, 2013 at 10:49:56AM +0100, Jakub Jelinek wrote: > > On Mon, Nov 25, 2013 at 10:44:57AM +0100, Marek Polacek wrote: > > > The PR was fixed by Jakub in r205283, this patch merely adds a > > > testcase for it. Passed ubsa

Re: [PATCH] Add testcase for PR59250

2013-11-25 Thread Marek Polacek
On Mon, Nov 25, 2013 at 10:49:56AM +0100, Jakub Jelinek wrote: > On Mon, Nov 25, 2013 at 10:44:57AM +0100, Marek Polacek wrote: > > The PR was fixed by Jakub in r205283, this patch merely adds a > > testcase for it. Passed ubsan testsuite for -m32/-m64. > > > > Ok for trunk? > > Can't you reduce

Re: [PATCH] Add testcase for PR59250

2013-11-25 Thread Jakub Jelinek
On Mon, Nov 25, 2013 at 10:44:57AM +0100, Marek Polacek wrote: > The PR was fixed by Jakub in r205283, this patch merely adds a > testcase for it. Passed ubsan testsuite for -m32/-m64. > > Ok for trunk? Can't you reduce it at least a little bit more? Like I doubt __attribute__ ((__visibility__ (