Re: [PATCH 01/13] recog: Increased max number of alternatives.

[Jeff Law <law at redhat dot com>]

On 05/11/2015 07:23 AM, Andreas Krebbel wrote:
> With the vector facility support z13 mov patterns have more than 30
> alternatives.
>
> gcc/
> 	* recog.h: Increase MAX_RECOG_ALTERNATIVES.
> ---
>   gcc/recog.h |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/recog.h b/gcc/recog.h
> index 8a38b26..4d8ca0c 100644
> --- a/gcc/recog.h
> +++ b/gcc/recog.h
> @@ -23,7 +23,7 @@ along with GCC; see the file COPYING3.  If not see
>   /* Random number that should be large enough for all purposes.  Also define
>      a type that has at least MAX_RECOG_ALTERNATIVES + 1 bits, with the extra
>      bit giving an invalid value that can be used to mean "uninitialized".  */
> -#define MAX_RECOG_ALTERNATIVES 30
> +#define MAX_RECOG_ALTERNATIVES 35
>   typedef unsigned int alternative_mask;
Yow! Won't think break on 32bit integer targets?

Which leads me to ponder how difficult it would be to catch this in the 
generators so that a port which had too many alternatives would at least 
issue a sensible error during the build process.

Jeff