[PATCH, testsuite] Add loop unrolling command line options for some test cases

Richard Guenther richard.guenther@gmail.com
Mon Sep 26 12:14:00 GMT 2011


On Mon, Sep 26, 2011 at 9:08 AM, Jiangning Liu <jiangning.liu@arm.com> wrote:
>> -----Original Message-----
>> From: Richard Guenther [mailto:richard.guenther@gmail.com]
>> Sent: Saturday, September 24, 2011 6:12 PM
>> To: Jiangning Liu
>> Cc: Mike Stump; gcc-patches@gcc.gnu.org; ro@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.liu@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?

Right.

>> > Thanks,
>> > -Jiangning
>> >
>> >> -----Original Message-----
>> >> From: Mike Stump [mailto:mikestump@comcast.net]
>> >> Sent: Thursday, September 22, 2011 3:10 AM
>> >> To: Jiangning Liu
>> >> Cc: gcc-patches@gcc.gnu.org; ro@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.
>> >
>> >
>> >
>> >
>> >
>
>
>
>
>



More information about the Gcc-patches mailing list