On Fri, Nov 4, 2011 at 7:44 AM, Richard Membarth
<[email protected]> wrote:
> Attached is a patch that adds support for pretty-printing attributes
> stored in the AST. Tblgen is used to generate the necessary code for
> pretty printing attributes as suggest by Peter in [1].
>
> Please let me know if this is ok.

+  // Pretty print this attribute.
+  virtual void printPretty(llvm::raw_ostream &OUT, ASTContext &C) const = 0;

Nit: "llvm::raw_ostream &OS".

I don't see any other issues; I'm not really that familiar with the
attribute TableGen bits, though.  I was hoping someone more familiar
with that would review.

-Eli
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to