[PATCH] Add warning capabilities in LLVM (backend part)

[Dmitri Gribenko]

On Fri, Nov 15, 2013 at 6:44 PM, Alp Toker <alp at nuanti.com> wrote:
>
> On 16/11/2013 02:26, Dmitri Gribenko wrote:
>>
>> On Fri, Nov 15, 2013 at 5:51 PM, Quentin Colombet <qcolombet at apple.com>
>> wrote:
>>>
>>> Finally the reporting API would be:
>>> void LLVMContext::report(DiagnosticInfo *, Kind, Severity)
>>> The default message would be given by toStr().
>>
>> I doubt that Clang will be using this API.
>
> My understanding is that the proposed LLVMContext::report() function is for
> LLVM to use. clang would then consume the results of these LLVM diagnostics
> via a callback, potentially converting them into clang diagnostics through
> the usual mechanism.

Ah, I see.  I thought that this was the default reporting API that the
fronted can use to just dump the diagnostic to stdout.  Sorry for
misunderstanding.

I would request to call it diag() for consistency with Clang, then.
Or diagnose() if you don't like abbreviations.  To me, report() sounds
like reporting a message to the user.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/