This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch: RTL loop optimizer - new target hook in doloop_valid_p
- From: Adrian Straetling <straetling at de dot ibm dot com>
- To: Richard Henderson <rth at redhat dot com>, gcc-patches at gcc dot gnu dot org, dje at gcc dot gnu dot org
- Cc: Ulrich dot Weigand at de dot ibm dot com
- Date: Thu, 19 May 2005 16:50:46 +0200
- Subject: Re: patch: RTL loop optimizer - new target hook in doloop_valid_p
- References: <20050518123021.GA30757@de.ibm.com> <20050518194818.GB7203@redhat.com>
On Wed, May 18, 2005 at 12:48:18PM -0700, Richard Henderson wrote:
> On Wed, May 18, 2005 at 02:30:21PM +0200, Adrian Straetling wrote:
> > * loop-doloop.c: Include "target.h".
> > (doloop_valid_p): Move tests to function in targhooks.c.
> > * target.h: (struct gcc_target): New target hook
> > "insn_valid_within_doloop".
> > * target-def.h: Define default value for "insn_valid_within_doloop".
> > (TARGET_INITIALIZER): Insert new target hook into initializer.
> > * targhooks.c: (default_insn_valid_within_doloop): New function.
> > * targhooks.h: Declare "default_insn_valid_within_doloop".
> > * hooks.c: (hook_bool_rtx_true): New function.
> > * hooks.h: Declare "hook_bool_rtx_true".
> > * doc/tm.texi: Add documentation for new target hook.
>
> Ok. I should add the same hook_bool_rtx_true for ia64...
> ...
Thank you for the quick review. Bootstrap succeeded on i686 as well and
showed no regressions.
However, please do not commit this patch. Ulrich Weigand has to do that for legal
reasons.
thanks,
Adrian