Re: Recording predicate information in all rtl gen* programs

["H.J. Lu" <hjl.tools@gmail.com>]

On Tue, Apr 12, 2011 at 5:28 AM, Richard Sandiford
<richard.sandiford@linaro.org> wrote:
> This patch is a prerequisite for:
>
>    http://gcc.gnu.org/ml/gcc-patches/2011-03/msg02168.html
>
> (approved but not yet applied, because I'd forgotten about this).
>
> At the moment, gen* programs that want predicate information need
> to process the DEFINE*_PREDICATE directives themselves.  They can then
> use routines in gensupport.c to record and retrieve the processed data.
>
> This patch moves the directive processing into gensupport.c too, so that
> all gen* programs have access to it.
>
> Tested on x86_64-linux-gnu and arm-linux-gnueabi.  OK to install?
>
> Richard
>
>
> gcc/
>        * genpreds.c (process_define_predicate): Move most processing
>        to gensupport.c.  Continue to validate the expression.
>        * genrecog.c (did_you_mean_codes, compute_predicate_codes)
>        (process_define_predicate): Move processing to gensupport.c.
>        (main): Remove DEFINE_PREDICATE and DEFINE_SPECIAL_PREDICATE cases.
>        * gensupport.c (did_you_mean_codes): Moved from genrecog.c.
>        (compute_predicate_codes): Moved from genrecog.c.  Add lineno
>        argument.
>        (valid_predicate_name_p): New function, split out from old
>        genpreds.c:process_define_predicate.
>        (process_define_predicate): New function, combining code from
>        old genpreds.c and genrecog.c functions.
>        (process_rtx): Call it for DEFINE_PREDICATE and
>        DEFINE_SPECIAL_PREDICATE.
>
>

I think your patch caused:

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48573

-- 
H.J.