This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix is_gimple_tmp_var properly
- From: Daniel Jacobowitz <drow at false dot org>
- To: Nathanael Nerode <neroden at twcny dot rr dot com>
- Cc: gcc-patches at gcc dot gnu dot org, kenner at vlsi1 dot ultra dot nyu dot edu
- Date: Mon, 16 Aug 2004 10:15:39 -0400
- Subject: Re: [PATCH] Fix is_gimple_tmp_var properly
- References: <20040814234326.GA2401@twcny.rr.com>
On Sat, Aug 14, 2004 at 07:43:26PM -0400, Nathanael Nerode wrote:
> * tree.h (struct tree_decl): Add gimple_tmp_var bit and
> macro to test it.
> * tree-gimple.c (is_gimple_tmp_var): Use the bit.
> * gimplify.c (create_tmp_var_raw): Set the bit.
Formatting again...
> + /* It has certain guaranteed properties due to the way we generated it. */
Two spaces after a full stop.
> - return (TREE_CODE (t) == VAR_DECL && DECL_ARTIFICIAL (t)
> + return (TREE_CODE (t) == VAR_DECL
> + && DECL_ARTIFICIAL (t) && DECL_GIMPLE_TMP_VAR (t)
> && !TREE_STATIC (t) && !DECL_EXTERNAL (t));
Eight spaces there should be a tab.
> }
>
> Index: tree.h
> ===================================================================
> RCS file: /cvs/gcc/gcc/gcc/tree.h,v
> retrieving revision 1.587
> diff -u -r1.587 tree.h
> --- tree.h 12 Aug 2004 14:34:01 -0000 1.587
> +++ tree.h 14 Aug 2004 01:41:01 -0000
> @@ -2167,6 +2167,13 @@
> #define DECL_POSSIBLY_INLINED(DECL) \
> FUNCTION_DECL_CHECK (DECL)->decl.possibly_inlined
>
> +/* Nonzero for a decl which is a GIMPLE temporary variable. These
> + are always DECL_ARTIFICAL, but also have other known and required
> + properties (which are slightly hard to describe). */
Two spaces.
--
Daniel Jacobowitz