Re: PING: [PATCH]: Fix -fbranch-probabilities

2011-08-29 Thread Richard Guenther
On Mon, Aug 29, 2011 at 1:59 PM, Christian Bruel wrote: > > > On 08/27/2011 02:04 AM, Jan Hubicka wrote: >>> >>> Hello, >>> >>> Could I have a review for the trivial patch posted in >>> http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01123.html >>> >>> -fprofile-use sets flag_branch_probabilities. >>

Re: PING: [PATCH]: Fix -fbranch-probabilities

2011-08-29 Thread Christian Bruel
On 08/27/2011 02:04 AM, Jan Hubicka wrote: Hello, Could I have a review for the trivial patch posted in http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01123.html -fprofile-use sets flag_branch_probabilities. But we should also be able to use -fbranch-probabilities on its own using the informat

Re: PING: [PATCH]: Fix -fbranch-probabilities

2011-08-26 Thread Jan Hubicka
> Hello, > > Could I have a review for the trivial patch posted in > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01123.html > > -fprofile-use sets flag_branch_probabilities. > > But we should also be able to use -fbranch-probabilities on its own > using the information generated by -fprofile-arc

PING: [PATCH]: Fix -fbranch-probabilities

2011-08-26 Thread Christian Bruel
Hello, Could I have a review for the trivial patch posted in http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01123.html -fprofile-use sets flag_branch_probabilities. But we should also be able to use -fbranch-probabilities on its own using the information generated by -fprofile-arcs, as documente

[PATCH]: Fix -fbranch-probabilities

2011-08-12 Thread Christian Bruel
Hello, -fbranch-probabilities fails to find the gcda information, because they are initialized only if flag_profile_use. The problem is easily observable by recompiling with -fbranch-probabilities instead of -fprofile-use (after profile information generation) This fails with "xxx.gcda not