On Sat, Jan 26, 2019 at 01:25:04AM +0000, Joseph Myers wrote:
> It's also broken the build of the glibc testsuite, e.g.:
> 
> ../time/time.h:88:15: error: mismatch in argument 1 type of built-in function 
> 'strftime'; expected 'char *' [-Werror=builtin-declaration-mismatch]
>    88 | extern size_t strftime (char *__restrict __s, size_t __maxsize,
> 
> (presence or absence of qualifiers on a parameter is not part of the 
> function type and should not be compared here).

There are several further issues, even after fixing the toplevel qualifiers on
arguments as well as return type + fixing up the testcases, there is another
problem, the code has the magic handling only for fileptr_type_node, but we
actually need handling for all builtin_structptr_types struct types,
otherwise e.g. for strftime we warn that the last argument should not be
const struct tm *, but const void *, when it actually should be const struct
tm *.

Working on it.

        Jakub

Reply via email to