Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2015-01-14 Thread Jeff Law
On 11/05/14 13:30, jb...@gmx.de wrote: With LTO enabled it runs just fine (which is the reason for the patch I suggested): This is definitely a testing framework problem. The profopt framework isn't clearing the "additional_whatever" variables. Presumably failure to clear is specific to a t

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2015-01-14 Thread Jeff Law
On 11/05/14 13:30, jb...@gmx.de wrote: "Jeff Law" : On 10/23/14 08:30, jb...@gmx.de wrote: "Jeff Law" : On 10/21/14 12:21, jb...@gmx.de wrote: "Jeff Law" : On 10/21/14 16:13, Haswell wrote: The additional source must have the same requirement crossmodule-indircall-1.c has. * cros

Re: Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-11-05 Thread jb999
"Jeff Law" : >On 10/23/14 08:30, jb...@gmx.de wrote: >> "Jeff Law" : >> >>> On 10/21/14 12:21, jb...@gmx.de wrote: "Jeff Law" : > On 10/21/14 16:13, Haswell wrote: >> The additional source must have the same requirement >> crossmodule-indircall-1.c has. >> >> * crossmodul

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-10-24 Thread Jeff Law
On 10/23/14 08:30, jb...@gmx.de wrote: "Jeff Law" : On 10/21/14 12:21, jb...@gmx.de wrote: "Jeff Law" : On 10/21/14 16:13, Haswell wrote: The additional source must have the same requirement crossmodule-indircall-1.c has. * crossmodule-indircall-1a.c: Add missing requirement. Why?

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-10-23 Thread jb999
"Jeff Law" : > On 10/21/14 12:21, jb...@gmx.de wrote: > > "Jeff Law" : > >> On 10/21/14 16:13, Haswell wrote: > >>> The additional source must have the same requirement > >>> crossmodule-indircall-1.c has. > >>> > >>> * crossmodule-indircall-1a.c: Add missing requirement. > >> Why? When used b

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-10-22 Thread Jeff Law
On 10/21/14 12:21, jb...@gmx.de wrote: "Jeff Law" : On 10/21/14 16:13, Haswell wrote: The additional source must have the same requirement crossmodule-indircall-1.c has. * crossmodule-indircall-1a.c: Add missing requirement. Why? When used by crossmodule-indircall-1.c we'll have alr

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-10-21 Thread jb999
"Jeff Law" : >On 10/21/14 16:13, Haswell wrote: >> The additional source must have the same requirement >> crossmodule-indircall-1.c has. >> >> * crossmodule-indircall-1a.c: Add missing requirement. >Why? When used by crossmodule-indircall-1.c we'll have already tested >the marker and when

Re: [PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-10-21 Thread Jeff Law
On 10/21/14 16:13, Haswell wrote: The additional source must have the same requirement crossmodule-indircall-1.c has. * crossmodule-indircall-1a.c: Add missing requirement. Why? When used by crossmodule-indircall-1.c we'll have already tested the marker and when used by itself, it doe

[PATCH] Add missing requirement to crossmodule-indircall-1a.c

2014-10-21 Thread Haswell
The additional source must have the same requirement crossmodule-indircall-1.c has. * crossmodule-indircall-1a.c: Add missing requirement. diff -Nur a/gcc/testsuite/gcc.dg/tree-prof/crossmodule-indircall-1a.c b/gcc/testsuite/gcc.dg/tree-prof/crossmodule-indircall-1a.c --- a/gcc/testsuit