Re: [patch,fortran] Revamp type enumerators

[Jerry DeLisle <jvdelisle at frontier dot com>]

On 10/17/2010 04:02 AM, Thomas Koenig wrote:
> Hi Jerry,
>
> > +/* Basic types.  BT_VOID is used by ISO C Binding so funcs like
> > c_f_pointer
> > +   can take any arg with the pointer attribute as a param.  These are
> > also
> > +   used in the run-time library for IO.  */
> >   typedef enum
> > -{
> > -  GFC_DTYPE_UNKNOWN = 0,
> > -  GFC_DTYPE_INTEGER,
> > -  /* TODO: recognize logical types.  */
> > -  GFC_DTYPE_LOGICAL,
> > -  GFC_DTYPE_REAL,
> > -  GFC_DTYPE_COMPLEX,
> > -  GFC_DTYPE_DERIVED,
> > -  GFC_DTYPE_CHARACTER
> > +{ BT_UNKNOWN = 0, BT_INTEGER, BT_LOGICAL, BT_CHARACTER, BT_REAL,
> > BT_COMPLEX,
> > +  BT_DERIVED, BT_CLASS, BT_PROCEDURE, BT_HOLLERITH, BT_VOID
> >   }
> > -dtype;
> > -
> > +bt;
>
> Isn't changing the values of the enums an ABI change?

I was trying to avoid that, but probably got carried away. How about I commit 
the change to the fortran-exp branch so that it gets merged into 4.7 when we 
plan to break ABI anyway?

Jerry