Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-24 Thread H.J. Lu
On Mon, Nov 24, 2014 at 10:41 AM, Paolo Carlini wrote: > Hi again, > > > On 11/24/2014 07:33 PM, Paolo Carlini wrote: >> >> Hi, >> >> On 11/12/2014 03:02 AM, H.J. Lu wrote: >>> >>> I am checking in this. >> >> Looks like devirt-42.C is failing again and reverting your tweak would fix >> it?!? > >

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-24 Thread Paolo Carlini
Hi again, On 11/24/2014 07:33 PM, Paolo Carlini wrote: Hi, On 11/12/2014 03:02 AM, H.J. Lu wrote: I am checking in this. Looks like devirt-42.C is failing again and reverting your tweak would fix it?!? Sorry, in fact now the line failing is "First type is base of second" 3, a different one.

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-24 Thread Paolo Carlini
Hi, On 11/12/2014 03:02 AM, H.J. Lu wrote: I am checking in this. Looks like devirt-42.C is failing again and reverting your tweak would fix it?!? Thanks, Paolo.

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-11 Thread H.J. Lu
On Tue, Nov 4, 2014 at 7:54 AM, Jiong Wang wrote: > > On 04/11/14 15:51, Paolo Carlini wrote: >> >> Hi, >> >> On 11/04/2014 04:37 PM, Martin Jambor wrote: >>> >>> Hi, >>> >>> since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing >>> because although the tested-for devirtualization doe

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-04 Thread Richard Biener
On Tue, Nov 4, 2014 at 4:57 PM, Martin Jambor wrote: > On Tue, Nov 04, 2014 at 04:47:18PM +0100, Richard Biener wrote: >> On Tue, Nov 4, 2014 at 4:37 PM, Martin Jambor wrote: >> > Hi, >> > >> > since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing >> > because although the tested-for

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-04 Thread Martin Jambor
On Tue, Nov 04, 2014 at 04:47:18PM +0100, Richard Biener wrote: > On Tue, Nov 4, 2014 at 4:37 PM, Martin Jambor wrote: > > Hi, > > > > since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing > > because although the tested-for devirtualization does happen, it is > > probably being done

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-04 Thread Jiong Wang
On 04/11/14 15:51, Paolo Carlini wrote: Hi, On 11/04/2014 04:37 PM, Martin Jambor wrote: Hi, since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing because although the tested-for devirtualization does happen, it is probably being done earlier and the string we are trying to match

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-04 Thread Paolo Carlini
Hi, On 11/04/2014 04:37 PM, Martin Jambor wrote: Hi, since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing because although the tested-for devirtualization does happen, it is probably being done earlier and the string we are trying to match is not emitted. But the important thing

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-04 Thread Richard Biener
On Tue, Nov 4, 2014 at 4:37 PM, Martin Jambor wrote: > Hi, > > since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing > because although the tested-for devirtualization does happen, it is > probably being done earlier and the string we are trying to match is > not emitted. But the imp