On 17/12/2019 17:47, Steve Kargl wrote:
On Tue, Dec 17, 2019 at 05:28:05PM +0000, Mark Eggleston wrote:
On 17/12/2019 17:06, Steve Kargl wrote:
On Tue, Dec 17, 2019 at 03:41:41PM +0000, Mark Eggleston wrote:
gcc/fortran/ChangeLog

       Mark Eggleston  <mark.eggles...@codethink.com>

       PR fortran/92896
       * array.c (walk_array_constructor): Replace call to cfg_convert_type
s/cfg_convert_type/gfc_convert_type

       with call to gfc_convert_type_warn with new argument set to true.
       (check_element_type): Replace call to cfg_convert_type with call to
       gfc_convert_type_warn with new argument set to true.
       * gfortran.h: Add argument "array" to gfc_convert_type_warn default
       value set to false.
Do all current uses of gfc_convert_type_warn need to be updated
to account for the new parameter?  That is, doesn't this introduce
a mismatch in the prototype and existing code?
I used a default value so all existing calls remain as they are and
default to false. So no mismatch.

% cat a.h
#ifndef _STDBOOL_H_
#include <stdbool.h>
#endif
float foo(int, float, bool tmp = false);
% cat a.c
#include "a.h"
void
bar(float x)
{
   int n;
   n = 1;
   x = foo(n, x);
}
% /usr/home/sgk/work/x/bin/gcc -Wall -c a.c
In file included from a.c:2:
a.h:1:32: error: expected ';', ',' or ')' before '=' token
     1 | float foo(int, float, bool tmp = false);
       |                                ^
a.c: In function 'bar':
a.c:8:7: warning: implicit declaration of function 'foo' 
[-Wimplicit-function-declaration]
     8 |   x = foo(n, x);
       |       ^~~

That's compiled with the C compiler, as I understand it the Fortran FE is in written in C++ and compiled using the C++ compiler even though the file extensions are .c.

regards,

Mark

--
https://www.codethink.co.uk/privacy.html

Reply via email to