Re: [google] Enable loop unroll/peel notes under -fopt-info

2011-11-02 Thread Xinliang David Li
Ok for google/main. 1) may be better to omit the const iteration count for complete unroll message 2) Instead of dumping loop header count, is it better to dump pre-header count -- it gives an idea of how often the loop is entered. The loop header count can be derived from loop average iteration a

Re: [google] Enable loop unroll/peel notes under -fopt-info

2011-11-02 Thread Xinliang David Li
On Tue, Nov 1, 2011 at 2:53 AM, Richard Guenther wrote: > On Tue, Nov 1, 2011 at 1:46 AM, Teresa Johnson wrote: >> This patch is for google-main only. >> >> Tested with bootstrap and regression tests. >> >> Print unroll and peel factors along with loop source position under >> -fopt-info. >> >>

Re: [google] Enable loop unroll/peel notes under -fopt-info

2011-11-01 Thread Teresa Johnson
Hi Richard, Once we have a uniform way to emit notes to either stderr or dump, as you and David had discussed in the earlier thread, we can merge these two messages. The advantage with the new messages, besides going to stderr, is that the source position information is being emitted since it is a

Re: [google] Enable loop unroll/peel notes under -fopt-info

2011-11-01 Thread Richard Guenther
On Tue, Nov 1, 2011 at 1:46 AM, Teresa Johnson wrote: > This patch is for google-main only. > > Tested with bootstrap and regression tests. > > Print unroll and peel factors along with loop source position under > -fopt-info. > > Teresa > > 2011-10-31   Teresa Johnson   > >        * common.opt (f