RE: r277134 - [GCC] Support for __final specifier

2016-07-29 Thread Keane, Erich via cfe-commits
...@intel.com>> Cc: cfe-commits <cfe-commits@lists.llvm.org<mailto:cfe-commits@lists.llvm.org>> Subject: Re: r277134 - [GCC] Support for __final specifier Mike, Thanks for letting me know. I committed a patch authored by Erich (CCed); Erich, could you, please, investigate? Yours,

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 <aizat...@google.com>; Keane, Erich <erich.ke...@intel.com> Cc: cfe-commits <cfe-commits@lists.llvm.org> Subject: Re: r277134 - [GCC] S

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: > >

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)

r277134 - [GCC] Support for __final specifier

2016-07-29 Thread Andrey Bokhanko via cfe-commits
Author: asbokhan Date: Fri Jul 29 05:42:48 2016 New Revision: 277134 URL: http://llvm.org/viewvc/llvm-project?rev=277134=rev Log: [GCC] Support for __final specifier As reported in bug 28473, GCC supports "final" functionality in pre-C++11 code using the __final keyword. Clang currently