Steve Kargl wrote:
On Wed, Mar 26, 2014 at 06:56:15PM +0100, Dominique Dhumieres wrote:
Updated patch. OK to commit?
OK.

I have a minor nit: can you add a @code{} around the second VOLATILE?

+Some Fortran compilers, including @command{g77}, let the user declare
+@code{COMMON} with the @code{VOLATILE} attribute. This is
+invalid standard Fortran syntax and is not supported by
+@command{gfortran}.  Note that @command{gfortran} accepts VOLATILE
+variables in COMMON blocks since revision 4.3.

Thanks for the patch - and welcome to the club of GCC contributors with commit access :-)

Tobias

Reply via email to