[cfe-commits] r145930 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDecl.cpp test/Misc/warning-flags.c test/Sema/builtins.c test/Sema/c89.c test/Sema/implicit-decl.c

Enea Zaffanella zaffanella at cs.unipr.it
Fri Jan 6 11:46:49 PST 2012


Il 06/12/2011 10:46, Hans Wennborg ha scritto:
> Author: hans
> Date: Tue Dec  6 03:46:12 2011
> New Revision: 145930
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=145930&view=rev
> Log:
> Suggest typo corrections for implicit function declarations.
> 
> A mistyped function call becomes an inmplicit function declaration in C.
> Suggest typo correction when one can be found.

[...]

> -def warn_builtin_unknown : Warning<"use of unknown builtin %0">, DefaultError;
> +def err_builtin_unknown : Error<"use of unknown builtin %0">;


Was this change (from warning to error) really needed to achieve what is
specified in the commit message?

We tried reverting from error to warning (mapped by default to Error,
see attached patch) and we observed no failure in the clang tests.

Having this as a warning allows clients to freely remap the diagnostic
as more appropriate for their application (e.g., ignore it).
Are there objections to committing this patch?

Enea.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: builtin-warn.patch
Type: text/x-patch
Size: 1324 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120106/51918e48/attachment.bin>


More information about the cfe-commits mailing list