On Jul 26, 2012, at 9:01 , Alexander Kornienko <[email protected]> wrote:

> Author: alexfh
> Date: Thu Jul 26 11:01:23 2012
> New Revision: 160784
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=160784&view=rev
> Log:
> Added -ast-dump-filter option to clang -cc1.
> 
> Modified:
>    cfe/trunk/include/clang/AST/DeclBase.h
>    cfe/trunk/include/clang/Driver/CC1Options.td
>    cfe/trunk/include/clang/Frontend/ASTConsumers.h
>    cfe/trunk/include/clang/Frontend/FrontendOptions.h
>    cfe/trunk/lib/AST/DeclPrinter.cpp
>    cfe/trunk/lib/AST/DumpXML.cpp
>    cfe/trunk/lib/Frontend/ASTConsumers.cpp
>    cfe/trunk/lib/Frontend/CompilerInvocation.cpp
>    cfe/trunk/lib/Frontend/FrontendActions.cpp
> 
> Modified: cfe/trunk/include/clang/AST/DeclBase.h
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/DeclBase.h?rev=160784&r1=160783&r2=160784&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/AST/DeclBase.h (original)
> +++ cfe/trunk/include/clang/AST/DeclBase.h Thu Jul 26 11:01:23 2012
> @@ -857,9 +857,8 @@
>   static void printGroup(Decl** Begin, unsigned NumDecls,
>                          raw_ostream &Out, const PrintingPolicy &Policy,
>                          unsigned Indentation = 0);
> -  LLVM_ATTRIBUTE_USED void dump() const;
> -  LLVM_ATTRIBUTE_USED void dumpXML() const;
> -  void dumpXML(raw_ostream &OS) const;
> +  LLVM_ATTRIBUTE_USED void dump(raw_ostream &Out = llvm::errs()) const;
> +  LLVM_ATTRIBUTE_USED void dumpXML(raw_ostream &OS = llvm::errs()) const;

The reason why we like having a dump() without a default argument is because 
debuggers don't usually respect default arguments. Please put it back!
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to