On Fri, Nov 4, 2016 at 8:50 AM, Serge Pavlov <sepavl...@gmail.com> wrote: > 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.
Ah, thank you! :-) ~Aaron > > 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
