On Sun, Jan 15, 2012 at 01:39:23PM -0500, Zydrunas Gimbutas wrote: > Hi, > > Everything seems to be ok in this patch, except for minor typos in > documentation: REAl -> REAL, alignement -> alignment.
Good catch. > It would also be nice to extend the type conversion facility to > include -fdefault-real-16, -fdefault-double-16, -fdefault-real-10, > -fdefault-double-10, and add conversions for logical types, to > complete the conversion table. These are not very commonly used > (except for -fdefault-real-16, -fdefault-double-16, which would > correspond to -r16 and -autodouble flags in some compilers), so we > should be fine at this time. Well, historically, -fdefault-real-8, -fdefault-integer-8, and -fdefault-double-8 were provided for backwards compatibility with g77's -r8, -d8, and -i8 options. There was no consideration for other compilers.* Thus, I would discourage these new options in favor of your -freal-N-real-M options. * Craig Burley may have included -r8, -i8, and -d8 to be compatible with other compilers, but I'm too lazy to go code spelunking. -- Steve
