This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, committed] Add ability to set the optimization options and on ix86 target options on a function specific basis
- From: Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>
- To: Michael Meissner <gnu at the-meissners dot org>, gcc-patches at gcc dot gnu dot org, karthikkumar at gmail dot com
- Date: Thu, 24 Jul 2008 07:20:43 +0200
- Subject: Re: [PATCH, committed] Add ability to set the optimization options and on ix86 target options on a function specific basis
- References: <20080723103750.GA9075@tiktok.the-meissners.org>
Hello, and a couple of nits:
* Michael Meissner wrote on Wed, Jul 23, 2008 at 12:37:51PM CEST:
> --- gcc/attribs.c (revision 138074)
> +++ gcc/attribs.c (working copy)
> @@ -33,6 +33,7 @@ along with GCC; see the file COPYING3.
> #include "target.h"
> #include "langhooks.h"
> #include "hashtab.h"
> +#include "c-common.h"
This needs adjustment of Makefile.in, iff you really need this include.
> --- gcc/targhooks.c (revision 138074)
> +++ gcc/targhooks.c (working copy)
> @@ -709,4 +709,38 @@ default_hard_regno_scratch_ok (unsigned
[...]
> +
> + /* If both caller and callee have attributes, assume that if the pointer is
> + different, the the two functions have different target options since
s/the the/the/
> + build_target_option_node uses a hash table for the options. */
> + else
> + ret = (callee_opts == caller_opts);
Cheers,
Ralf