> -----Original Message----- > From: Richard Guenther [mailto:richard.guent...@gmail.com] > Sent: Saturday, September 24, 2011 6:12 PM > To: Jiangning Liu > Cc: Mike Stump; gcc-patches@gcc.gnu.org; r...@cebitec.uni-bielefeld.de > Subject: Re: [PATCH, testsuite] Add loop unrolling command line options > for some test cases > > On Thu, Sep 22, 2011 at 6:22 AM, Jiangning Liu <jiangning....@arm.com> > wrote: > > Hi Mike, > > > > OK. I will wait 24 more hours. If no objections by then, I will get > it > > checked into trunk. > > I don't think you need -funroll-loops though. >
The intention of those test cases are not hurt if -funroll-loops is added, right? > > Thanks, > > -Jiangning > > > >> -----Original Message----- > >> From: Mike Stump [mailto:mikest...@comcast.net] > >> Sent: Thursday, September 22, 2011 3:10 AM > >> To: Jiangning Liu > >> Cc: gcc-patches@gcc.gnu.org; r...@cebitec.uni-bielefeld.de > >> Subject: Re: [PATCH, testsuite] Add loop unrolling command line > options > >> for some test cases > >> > >> On Sep 21, 2011, at 1:22 AM, Jiangning Liu wrote: > >> > The fix is to explicitly turn on loop unroll and set max-unroll- > times > >> to 8, > >> > which is larger than the unrolling times being detected in the > cases. > >> > >> Sounds reasonable to me. Ok, though, do watch for any comments by > >> people that know more than I. > > > > > > > > > >
