RE: r277134 - [GCC] Support for __final specifier

2016-07-29 Thread Keane, Erich via cfe-commits
Fixed in: r277206 after review in https://reviews.llvm.org/D22970 From: Keane, Erich Sent: Friday, July 29, 2016 11:58 AM To: 'Andrey Bokhanko' ; Mike Aizatsky Cc: cfe-commits Subject: RE: r277134 - [GCC] Support for __final specifier Sure, looking into it now. From: Andre

RE: r277134 - [GCC] Support for __final specifier

2016-07-29 Thread Keane, Erich via cfe-commits
Sure, looking into it now. From: Andrey Bokhanko [mailto:andreybokha...@gmail.com] Sent: Friday, July 29, 2016 11:57 AM To: Mike Aizatsky ; Keane, Erich Cc: cfe-commits Subject: Re: r277134 - [GCC] Support for __final specifier Mike, Thanks for letting me know. I committed a patch authored by

Re: r277134 - [GCC] Support for __final specifier

2016-07-29 Thread Andrey Bokhanko via cfe-commits
Mike, Thanks for letting me know. I committed a patch authored by Erich (CCed); Erich, could you, please, investigate? Yours, Andrey On Fri, Jul 29, 2016 at 9:35 PM, Mike Aizatsky wrote: > Andrey, > > This change breaks the bot: > > http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fa

Re: r277134 - [GCC] Support for __final specifier

2016-07-29 Thread Mike Aizatsky via cfe-commits
Andrey, This change breaks the bot: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/15093 ==15371==WARNING: MemorySanitizer: use-of-uninitialized-value #0 0x733784d in clang::Parser::ParseOptionalCXX11VirtSpecifierSeq(clang::VirtSpecifiers&, bool, clang::SourceLocation)