On Mon, Apr 8, 2013 at 5:38 PM, Tobias Burnus <bur...@net-b.de> wrote: > Dear all, > > attached is an updated version of the patch, which address the raised issues > and some minor problems and omissions I found. > > OK for the trunk?
+For Boolean (@code{LOGICAL}) arguments, please note that GCC expects +only the integer value 0 and 1. If a GNU Fortran @code{LOGICAL} +variable contains another integer value, the result is undefined. +As some other Fortran compilers use @math{-1} for @code{.TRUE.}, +extra care has to be taken -- such as passing the value as +@code{INTEGER}. (The same value restriction also applies to other +front ends of GCC, e.g. to GCC's C99 compiler for @code{_Bool} +or GCC's Ada compiler for @code{Complex}.) Presumably you meant Ada's @code{Bool} (or whatever the Ada boolean type is called)? Ok with that change. -- Janne Blomqvist