<div dir="rtl"><div dir="ltr">Committed clang part in r259232, LLVM part in r259240.</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2016-01-29 21:22 GMT+02:00 Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Jan 29, 2016 at 11:13 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
> OK.  There are more un-annotated dump() methods such as<br>
> TemplateName::dump(), NestedNameSpecifier::dump(), ASTReader::dump(),<br>
> ModuleMap::dump(), MacroInfo::dump(), ...<br>
><br>
> Annotate all dump() methods LLVM_DUMP_METHOD ?<br>
<br>
</span>SGTM<br>
<div class="HOEnZb"><div class="h5"><br>
> 2016-01-29 19:37 GMT+02:00 Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>>:<br>
>><br>
>> On 29 Jan 2016 5:50 a.m., "Yaron Keren via cfe-commits"<br>
>> <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br>
>> ><br>
>> > Author: yrnkrn<br>
>> > Date: Fri Jan 29 07:46:15 2016<br>
>> > New Revision: 259192<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=259192&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=259192&view=rev</a><br>
>> > Log:<br>
>> > Implement TemplateArgument::dump() method for debugging, patterned after<br>
>> > TemplateName::dump().<br>
>> ><br>
>> ><br>
>> > Modified:<br>
>> >     cfe/trunk/include/clang/AST/TemplateBase.h<br>
>> >     cfe/trunk/lib/AST/TemplateBase.cpp<br>
>> ><br>
>> > Modified: cfe/trunk/include/clang/AST/TemplateBase.h<br>
>> > URL:<br>
>> > <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/TemplateBase.h?rev=259192&r1=259191&r2=259192&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/TemplateBase.h?rev=259192&r1=259191&r2=259192&view=diff</a><br>
>> ><br>
>> > ==============================================================================<br>
>> > --- cfe/trunk/include/clang/AST/TemplateBase.h (original)<br>
>> > +++ cfe/trunk/include/clang/AST/TemplateBase.h Fri Jan 29 07:46:15 2016<br>
>> > @@ -354,6 +354,12 @@ public:<br>
>> >    /// \brief Print this template argument to the given output stream.<br>
>> >    void print(const PrintingPolicy &Policy, raw_ostream &Out) const;<br>
>> ><br>
>> > +  /// \brief Debugging aid that dumps the template argument.<br>
>> > +  void dump(raw_ostream &Out) const;<br>
>> > +<br>
>> > +  /// \brief Debugging aid that dumps the template argument to standard<br>
>> > error.<br>
>> > +  void dump() const;<br>
>> > +<br>
>> >    /// \brief Used to insert TemplateArguments into FoldingSets.<br>
>> >    void Profile(llvm::FoldingSetNodeID &ID, const ASTContext &Context)<br>
>> > const;<br>
>> >  };<br>
>> ><br>
>> > Modified: cfe/trunk/lib/AST/TemplateBase.cpp<br>
>> > URL:<br>
>> > <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/TemplateBase.cpp?rev=259192&r1=259191&r2=259192&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/TemplateBase.cpp?rev=259192&r1=259191&r2=259192&view=diff</a><br>
>> ><br>
>> > ==============================================================================<br>
>> > --- cfe/trunk/lib/AST/TemplateBase.cpp (original)<br>
>> > +++ cfe/trunk/lib/AST/TemplateBase.cpp Fri Jan 29 07:46:15 2016<br>
>> > @@ -415,6 +415,15 @@ void TemplateArgument::print(const Print<br>
>> >    }<br>
>> >  }<br>
>> ><br>
>> > +void TemplateArgument::dump(raw_ostream &Out) const {<br>
>> > +  LangOptions LO; // FIXME! see also TemplateName::dump().<br>
>> > +  LO.CPlusPlus = true;<br>
>> > +  LO.Bool = true;<br>
>> > +  print(PrintingPolicy(LO), Out);<br>
>> > +}<br>
>> > +<br>
>> > +void TemplateArgument::dump() const { dump(llvm::errs()); }<br>
>><br>
>> This should be annotated with LLVM_DUMP_METHOD.<br>
>><br>
>> > +<br>
>> ><br>
>> > //===----------------------------------------------------------------------===//<br>
>> >  // TemplateArgumentLoc Implementation<br>
>> ><br>
>> > //===----------------------------------------------------------------------===//<br>
>> ><br>
>> ><br>
>> > _______________________________________________<br>
>> > cfe-commits mailing list<br>
>> > <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>