This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Support for 'B' in RTL format strings
- From: Andreas Jaeger <aj at suse dot de>
- To: Jan Hubicka <jh at suse dot cz>
- Cc: gcc-patches at gcc dot gnu dot org, rth at cygnus dot com,gcc-pdo at atrey dot karlin dot mff dot cuni dot cz
- Date: Thu, 30 May 2002 22:02:57 +0200
- Subject: Re: Support for 'B' in RTL format strings
- References: <20020530174930.GG9581@atrey.karlin.mff.cuni.cz>
Jan Hubicka <jh@suse.cz> writes:
> Hi,
> I am working on patch to replace BLOCK_FOR_INSN array by field in INSN itself
> and I've noticed that we already do have kind of support for pointers to BBs in
> RTL structure by BBDEF accessor macro, but it requires code 'B' not documented
> nor supported by rype_from_format. I think it is cleaner than use '0', so here
> are the missing bits.
>
> Regtested/bootstrapped i386.
>
> Thu May 30 19:46:52 CEST 2002 Jan Hubicka <jh@suse.cz>
> * gengenrtl.c (type_from_format, accessor_from_format): Support 'B'.
> * rtl.texi: Document 'B'
> Index: gengenrtl.c
> ===================================================================
> RCS file: /cvs/gcc/egcs/gcc/gengenrtl.c,v
> retrieving revision 1.54
> diff -c -3 -p -r1.54 gengenrtl.c
> *** gengenrtl.c 19 May 2002 06:04:21 -0000 1.54
> --- gengenrtl.c 30 May 2002 17:46:39 -0000
> *************** type_from_format (c)
> *** 81,86 ****
> --- 81,88 ----
> return "struct bitmap_head_def *"; /* bitmap - typedef not available */
> case 't':
> return "union tree_node *"; /* tree - typedef not available */
> + case 'B':
> + return "struct basic_block_def *"; /* basic block - typedef not available */
> default:
> abort ();
> }
> *************** accessor_from_format (c)
> *** 115,122 ****
> case 't':
> return "XTREE";
>
> ! default:
> ! abort ();
> }
> }
>
> --- 117,124 ----
> case 't':
> return "XTREE";
>
> ! case 'B':
> ! return "XBBDEF";
You're removing the default case. Is this really correct?
Andreas
--
Andreas Jaeger
SuSE Labs aj@suse.de
private aj@arthur.inka.de
http://www.suse.de/~aj