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] Follow references in dependency checking


Am Montag, den 02.08.2010, 08:35 +0200 schrieb Tobias Burnus:
> Thomas Koenig wrote:
> > > +  while ((numref = (r1 != NULL) + (r2 != NULL)) > 0)
> > > +    {
> > > +
> > > +
> > > That's ugly. Can't you simply use the following?
> >
> > Changed to something more becoming.  You were right that this was
> > ugly :-)
> 
> Thanks. Can you also remove the - now obsolete - comment, which
> explains the different values of "numref"?

Done.

> > Currently regtesting.  OK if this passes?
> 
> OK. Thanks for the patch!

Sending        fortran/ChangeLog
Sending        fortran/dependency.c
Sending        fortran/dependency.h
Sending        fortran/frontend-passes.c
Sending        testsuite/ChangeLog
Adding         testsuite/gfortran.dg/character_comparison_2.f90
Adding         testsuite/gfortran.dg/character_comparison_3.f90
Adding         testsuite/gfortran.dg/dependency_28.f90
Transmitting file data ........
Committed revision 162824.

Thanks for the review!

	Thomas




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