This following commit r285981 (Enhancement to test for -ast-print tests) tests this change. Without it the test 'Coverage/ast-printing.c' fails on Windows.
Thanks, --Serge 2016-11-04 19:09 GMT+07:00 Aaron Ballman <aa...@aaronballman.com>: > On Fri, Nov 4, 2016 at 2:03 AM, Serge Pavlov via cfe-commits > <cfe-commits@lists.llvm.org> wrote: > > Author: sepavloff > > Date: Fri Nov 4 01:03:34 2016 > > New Revision: 285979 > > > > URL: http://llvm.org/viewvc/llvm-project?rev=285979&view=rev > > Log: > > Do not print enum underlying type if language is not C++11 > > > > Output generated by option '-ast-print' must not contains enum > > base type specifications if source language does not include C++11. > > > > Modified: > > cfe/trunk/lib/AST/DeclPrinter.cpp > > Test case? > > ~Aaron > > > > > Modified: cfe/trunk/lib/AST/DeclPrinter.cpp > > URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ > DeclPrinter.cpp?rev=285979&r1=285978&r2=285979&view=diff > > ============================================================ > ================== > > --- cfe/trunk/lib/AST/DeclPrinter.cpp (original) > > +++ cfe/trunk/lib/AST/DeclPrinter.cpp Fri Nov 4 01:03:34 2016 > > @@ -407,7 +407,7 @@ void DeclPrinter::VisitEnumDecl(EnumDecl > > } > > Out << *D; > > > > - if (D->isFixed()) > > + if (D->isFixed() && D->getASTContext().getLangOpts().CPlusPlus11) > > Out << " : " << D->getIntegerType().stream(Policy); > > > > if (D->isCompleteDefinition()) { > > > > > > _______________________________________________ > > cfe-commits mailing list > > cfe-commits@lists.llvm.org > > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
