[lld] r203102 - [Mips] Replace "virtual" by "override" in member function declarations

Duncan P. N. Exon Smith dexonsmith at apple.com
Thu Mar 6 06:45:20 PST 2014


On 2014 Mar 6, at 02:29, Simon Atanasyan <simon at atanasyan.com> wrote:

> Author: atanasyan
> Date: Thu Mar  6 04:29:45 2014
> New Revision: 203102
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=203102&view=rev
> Log:
> [Mips] Replace "virtual" by "override" in member function declarations
> where it is appropriate.
> 
> Modified:
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsDynamicLibraryWriter.h
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsDynamicTable.h
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsExecutableWriter.h
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsLinkingContext.h
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsRelocationHandler.h
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsRelocationPass.cpp
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsSectionChunks.h
>    lld/trunk/lib/ReaderWriter/ELF/Mips/MipsTargetHandler.h
> 
> Modified: lld/trunk/lib/ReaderWriter/ELF/Mips/MipsTargetHandler.h
> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/ELF/Mips/MipsTargetHandler.h?rev=203102&r1=203101&r2=203102&view=diff
> ==============================================================================
> @@ -72,17 +72,17 @@ class MipsTargetHandler final
> public:
>   MipsTargetHandler(MipsLinkingContext &context);
> 
> -  virtual MipsTargetLayout<Mips32ElELFType> &getTargetLayout() {
> +  MipsTargetLayout<Mips32ElELFType> &getTargetLayout() override {
>     return *(_mipsTargetLayout.get());
>   }
> 
> -  virtual const MipsTargetRelocationHandler &getRelocationHandler() const {
> +  const MipsTargetRelocationHandler &getRelocationHandler() const override {
>     return *(_mipsRelocationHandler.get());
>   }
> 
> -  virtual std::unique_ptr<Writer> getWriter();
> +  std::unique_ptr<Writer> getWriter() override;
> 
> -  virtual void registerRelocationNames(Registry &registry);
> +  void registerRelocationNames(Registry &registry) override ;

Whitespace nitpick: you have a space between "override" and ";".




More information about the llvm-commits mailing list