This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch, fortran, 4.5] F2003: simplifiers for array transformations


On Sat, Jan 03, 2009 at 05:54:37PM +0100, Daniel Franke wrote:
> 
> 	PR fortran/29962
> 	* gfortran.h (gfc_wide_strcpy): New prototype.
> 	* scanner.c (wide_strcpy): Renamed to ...
> 	(gfc_wide_strcpy): ... this. Fixed all callers.
> 	* simplify.c (init_result_expr): New.
> 	(trans_result): New.

Daniel, I've only skimmed over the diff, but noticed the 
names of functions that start with trans_*.  Traditionally,
functions that start with trans_* are located in the files
trans-*.c and are used to convert gfortran's black and red
trees to the TREE-SSA form.  Any chance that you would 
consider renaming the functions to xform_* or transform_*.
Note, this isn't a show stopper request, just something
to keep a clean segregation in the code.

-- 
Steve


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]