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

Alp Toker alp at nuanti.com
Fri Nov 15 18:44:22 PST 2013

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.

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

Quentin, did I get this right?


>    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

the browser experts

More information about the llvm-commits mailing list