[lld] r205644 - This is yet another case clang-modernize failed to add "override".

Rui Ueyama ruiu at google.com
Fri Apr 4 12:18:00 PDT 2014


Author: ruiu
Date: Fri Apr  4 14:17:59 2014
New Revision: 205644

URL: http://llvm.org/viewvc/llvm-project?rev=205644&view=rev
Log:
This is yet another case clang-modernize failed to add "override".

Modified:
    lld/trunk/lib/ReaderWriter/ELF/Atoms.h

Modified: lld/trunk/lib/ReaderWriter/ELF/Atoms.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/ELF/Atoms.h?rev=205644&r1=205643&r2=205644&view=diff
==============================================================================
--- lld/trunk/lib/ReaderWriter/ELF/Atoms.h (original)
+++ lld/trunk/lib/ReaderWriter/ELF/Atoms.h Fri Apr  4 14:17:59 2014
@@ -747,10 +747,9 @@ public:
 
 #ifndef NDEBUG
   StringRef name() const override { return _name; }
-
   std::string _name;
 #else
-  virtual StringRef name() const { return ""; }
+  StringRef name() const override { return ""; }
 #endif
 };
 
@@ -781,10 +780,9 @@ public:
 
 #ifndef NDEBUG
   StringRef name() const override { return _name; }
-
   std::string _name;
 #else
-  virtual StringRef name() const { return ""; }
+  StringRef name() const override { return ""; }
 #endif
 };
 
@@ -899,10 +897,9 @@ public:
 
 #ifndef NDEBUG
   StringRef name() const override { return _name; }
-
   std::string _name;
 #else
-  virtual StringRef name() const { return ""; }
+  StringRef name() const override { return ""; }
 #endif
 };
 





More information about the llvm-commits mailing list