[patch,fortran] Revamp type enumerators

Jerry DeLisle jvdelisle@frontier.com
Sun Oct 17 14:14:00 GMT 2010


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



More information about the Fortran mailing list