Re: [Fortran, DRAFT patch] PR 46321 - [OOP] Polymorphic deallocation

[Tobias Burnus <burnus at net-b dot de>]

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.