public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Re: [Patch, fortran] [0/4] gfc_ss structs initialization small refactoring
@ 2011-09-06  7:23 Tobias Burnus
  2011-09-08 15:59 ` Mikael Morin
  0 siblings, 1 reply; 3+ messages in thread
From: Tobias Burnus @ 2011-09-06  7:23 UTC (permalink / raw)
  To: Mikael Morin, fortran, gcc-patches

http://gcc.gnu.org/ml/fortran/2011-08/threads.html#00264

Mikael Morin wrote:
> the 4 follow-up patches try to refactor some common code
> initializing gfc_ss structs.
> Regression-tested (the 4 patches together only) on x86_64-freebsd8.2.
> OK for trunk?

The patch set is OK. I think it is a nice clean up and might
prevent future bugs due to incomplete initialization.

Sorry for the slow review.

Tobias

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [Patch, fortran] [0/4] gfc_ss structs initialization small refactoring
@ 2011-08-31 21:39 Mikael Morin
  0 siblings, 0 replies; 3+ messages in thread
From: Mikael Morin @ 2011-08-31 21:39 UTC (permalink / raw)
  To: gfortran, GCC patches

[-- Attachment #1: Type: text/plain, Size: 493 bytes --]

Hello,

the 4 follow-up patches try to refactor some common code initializing gfc_ss
structs.
Regression-tested (the 4 patches together only) on x86_64-freebsd8.2.
OK for trunk?

Mikael

 trans-array.c     |  181 ++++++++++++++++++++++++-----------------------------
 trans-array.h     |    6 ++
 trans-expr.c      |   45 ++++---------
 trans-intrinsic.c |   24 +-------
 trans-io.c        |    8 +--
 trans-stmt.c      |   30 +++-------
 6 files changed, 114 insertions(+), 180 deletions(-)


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-09-08 15:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-09-06  7:23 [Patch, fortran] [0/4] gfc_ss structs initialization small refactoring Tobias Burnus
2011-09-08 15:59 ` Mikael Morin
  -- strict thread matches above, loose matches on Subject: below --
2011-08-31 21:39 Mikael Morin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).