This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [rfa, spu] Fix setting of PARAM_MAX_COMPLETELY_PEEL_TIMES.
- From: trevor_smigiel at playstation dot sony dot com
- To: Ulrich Weigand <uweigand at de dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 10 Jul 2007 11:54:20 -0700
- Subject: Re: [rfa, spu] Fix setting of PARAM_MAX_COMPLETELY_PEEL_TIMES.
- References: <200707101357.l6ADvAnr005316@d12av02.megacenter.de.ibm.com>
> OK for mainline?
Yes.
Trevor
>
> ChangeLog:
>
> * spu.c (spu_optimization_options): Remove setting of parameter
> PARAM_MAX_COMPLETELY_PEEL_TIMES.
> (spu_override_options): Move it here.
>
> Index: gcc/config/spu/spu.c
> ===================================================================
> --- gcc/config/spu/spu.c (revision 126416)
> +++ gcc/config/spu/spu.c (working copy)
> @@ -266,11 +266,6 @@
> void
> spu_optimization_options (int level ATTRIBUTE_UNUSED, int size ATTRIBUTE_UNUSED)
> {
> - /* Small loops will be unpeeled at -O3. For SPU it is more important
> - to keep code small by default. */
> - if (!flag_unroll_loops && !flag_peel_loops)
> - PARAM_VALUE (PARAM_MAX_COMPLETELY_PEEL_TIMES) = 1;
> -
> /* Override some of the default param values. With so many registers
> larger values are better for these params. */
> MAX_PENDING_LIST_LENGTH = 128;
> @@ -286,6 +281,12 @@
> void
> spu_override_options (void)
> {
> + /* Small loops will be unpeeled at -O3. For SPU it is more important
> + to keep code small by default. */
> + if (!flag_unroll_loops && !flag_peel_loops
> + && !PARAM_SET_P (PARAM_MAX_COMPLETELY_PEEL_TIMES))
> + PARAM_VALUE (PARAM_MAX_COMPLETELY_PEEL_TIMES) = 1;
> +
> flag_omit_frame_pointer = 1;
>
> if (align_functions < 8)
> --
> Dr. Ulrich Weigand
> GNU Toolchain for Linux on System z and Cell BE
> Ulrich.Weigand@de.ibm.com