[PATCH] Add warning capabilities in LLVM (backend part), Take 2
hfinkel at anl.gov
Wed Dec 18 07:01:30 PST 2013
----- Original Message -----
> From: "Tobias Grosser" <tobias at grosser.es>
> To: "Quentin Colombet" <qcolombet at apple.com>
> Cc: reviews+D2376+public+8c083b1648e3e7b6 at llvm-reviews.chandlerc.com, "Chandler Carruth" <chandlerc at gmail.com>,
> "llvm-commits" <llvm-commits at cs.uiuc.edu>
> Sent: Wednesday, December 18, 2013 4:57:18 AM
> Subject: Re: [PATCH] Add warning capabilities in LLVM (backend part), Take 2
> On 12/16/2013 06:41 PM, Quentin Colombet wrote:
> > Hi Tobias,
> > On Dec 14, 2013, at 1:20 AM, Tobias Grosser <tobias at grosser.es>
> > wrote:
> >> On 12/12/2013 06:40 PM, Quentin Colombet wrote:
> >>> - Suppress DiagnosticInfoOther class.
> >>> - Add DK_FirstPluginKind and the related
> >>> getNextAvailablePluginKind function.
> >>> Hi dblaikie, rengolin, chandlerc, echristo,
> >>> http://llvm-reviews.chandlerc.com/D2376
> >> Hi Quentin,
> >> I just wanted to say that I did a simple test with this in Polly
> >> (as a plugin), and it seems to work as expected.
> > I am glad it worked for you!
> >> I found one minor issue:
> >> - The return type of getNextAvailablePluginDiagnosticKind should
> >> be
> >> DiagnosticKind, not int.
> > Well, technically, the returned type is not DiagnosticKind, as the
> > returned value is not a valid value in the statically known
> > DiagnosticKind enum.
> I added a missing namespace to the implementation of
> getNextAvailablePluginDiagnosticKind in r197556.
> I also used this interface to implement the first diagnostic in Polly
> I also saw you started to submit patches to clang. For plugin
> it would be amazing if a frontend could optionally register a
> that demangels function names. Like this, the plugin default printers
> could return something sensible.
Yep; we'll certainly do this (the design on the LLVM side is already in place).
> Thanks again,
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory
More information about the llvm-commits