[PATCH] Add diagnostic capabilities in LLVM (frontend part)

[Hal Finkel]

----- Original Message -----
> From: "Quentin Colombet" <qcolombet at apple.com>
> To: hfinkel at anl.gov, qcolombet at apple.com
> Cc: cfe-commits at cs.uiuc.edu
> Sent: Wednesday, January 22, 2014 5:44:32 PM
> Subject: Re: [PATCH] Add diagnostic capabilities in LLVM (frontend part)
> 
> 
>   Ping?
> 
>   @Dmitri Gribenko
>   The name of the added warning is clearly debatable and in
>   particular I do not have on strong opinion on them (especially
>   regarding the backend- prefix).
>   I did not find any equivalent for the inline-asm diagnostic in GCC,
>   thus there is nothing to match here AFAICT.
>   Regarding the stack-size thing, the closest thing in GCC is
>   frame-larger-than. Since we cannot, AFAIK, set an integer value to
>   a warning in clang, we cannot match this for now.
> 
>   @Hal
>   I do not have a good answer for the demangling of function’s name
>   in the reporting. I believe this could be addressed later and in
>   probably a better way by front-end people.
>   In other words, we can make a RFC for this after this specific
>   plumbing lands.
>   The bottom line is, I do not think we should hold on this patch
>   because of that.

Okay. Fine by me.

 -Hal

> 
>   Thanks for your feedbacks.
> 
>   -Quentin
> 
> http://llvm-reviews.chandlerc.com/D2430
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory