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

[Alp Toker]

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.

Dmitri,

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.

So LLVMContext::report() isn't for clang to use in the first place.

Quentin, did I get this right?

Alp.


>    For example, Clang has a
> feature to serialize all diagnostics to a file, and the easiest way to
> make it work with this feature to just let all diagnostics flow
> through the usual diagnostics engine.
>
> Dmitri
>

-- 
http://www.nuanti.com
the browser experts