[Fortran, DRAFT patch] PR 46321 - [OOP] Polymorphic deallocation
Tobias Burnus
burnus@net-b.de
Sat Jun 2 17:36:00 GMT 2012
Janus Weil wrote:
> The patch actually gives a few warnings:
Looking at those warnings, they seem to be valid C++ but invalid C89. As
Stages 2 and 3 are, by default, compiled by C++, I assume that
Alessandro does not see those.
By contrast, I assume that you (Janus) build GCC with the C compiler,
i.e. you configure with --disable-build-poststage1-with-cxx.
Thus, a default boot strap, shouldn't fail. Nonetheless, it is useful
to keep compatibility with C and bootstrapping with
--disable-build-poststage1-with-cxx.Hence, the warnings should be fixed.
(Bootstrapping implies -Werror.)
Tobias
> /home/jweil/gcc48/trunk/gcc/fortran/class.c: In function
> ‘gfc_find_derived_vtab’:
> /home/jweil/gcc48/trunk/gcc/fortran/class.c:912:8: warning: ISO C90
> forbids mixed declarations and code [-pedantic]
> /home/jweil/gcc48/trunk/gcc/fortran/class.c:932:7: warning: C++ style
> comments are not allowed in ISO C90 [enabled by default]
> /home/jweil/gcc48/trunk/gcc/fortran/class.c:932:7: warning: (this will
> be reported only once per input file) [enabled by default]
>
> (and similar things in trans-stmt.c). You should definitely fix those.
> Although a non-buildstrap build still works with those warnings, a
> full bootstrap will fail.
More information about the Gcc-patches
mailing list