On Jan 6, 2012, at 11:46 AM, Enea Zaffanella wrote:
> 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?
Please go ahead and commit.
- Doug
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
