r285979 - Do not print enum underlying type if language is not C++11

[Aaron Ballman via cfe-commits]

On Fri, Nov 4, 2016 at 8:50 AM, Serge Pavlov <sepavloff at 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 <aaron at aaronballman.com>:
>>
>> On Fri, Nov 4, 2016 at 2:03 AM, Serge Pavlov via cfe-commits
>> <cfe-commits at 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 at lists.llvm.org
>> > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
>