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. > 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. > > > > >