r231053 - Remove obsolete FIXME. override on definitions is fine.

Nico Weber nicolasweber at gmx.de
Mon Mar 2 19:31:15 PST 2015


Author: nico
Date: Mon Mar  2 21:31:15 2015
New Revision: 231053

URL: http://llvm.org/viewvc/llvm-project?rev=231053&view=rev
Log:
Remove obsolete FIXME. override on definitions is fine.

Modified:
    cfe/trunk/test/Parser/cxx0x-override-control-keywords.cpp

Modified: cfe/trunk/test/Parser/cxx0x-override-control-keywords.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/cxx0x-override-control-keywords.cpp?rev=231053&r1=231052&r2=231053&view=diff
==============================================================================
--- cfe/trunk/test/Parser/cxx0x-override-control-keywords.cpp (original)
+++ cfe/trunk/test/Parser/cxx0x-override-control-keywords.cpp Mon Mar  2 21:31:15 2015
@@ -10,10 +10,8 @@ struct S : Base {
   virtual void override() override;
 };
 
-struct T {
-  // virt-specifier-seq is only valid in member-declarators, and a function definition is not a member-declarator.
-  // FIXME: This currently doesn't work.
-  // virtual void f() const override { } 
+struct T : Base {
+  virtual void override() override { } 
 };
 
 struct override;





More information about the cfe-commits mailing list