[all-commits] [llvm/llvm-project] a19461: [NFC] Add 'override' keyword where missing in incl...

kepler-5 via All-commits all-commits at lists.llvm.org
Tue Jul 14 09:49:36 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a19461d9e114bea99186a4f0ea092a650a1650c2
      https://github.com/llvm/llvm-project/commit/a19461d9e114bea99186a4f0ea092a650a1650c2
  Author: Logan Smith <logan.r.smith0 at gmail.com>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/InstructionPrecedenceTracking.h
    M llvm/include/llvm/Analysis/MustExecute.h
    M llvm/include/llvm/Demangle/MicrosoftDemangleNodes.h
    M llvm/include/llvm/IR/DiagnosticInfo.h
    M llvm/include/llvm/MC/MCELFObjectWriter.h
    M llvm/include/llvm/MC/MCMachObjectWriter.h
    M llvm/include/llvm/MC/MCWasmObjectWriter.h
    M llvm/include/llvm/MC/MCWinCOFFObjectWriter.h
    M llvm/include/llvm/Object/Error.h
    M llvm/include/llvm/Support/FormatAdapters.h
    M llvm/lib/Analysis/LazyValueInfo.cpp
    M llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
    M llvm/lib/Target/AMDGPU/MCTargetDesc/R600MCCodeEmitter.cpp
    M llvm/lib/Target/AVR/AVRFrameLowering.cpp
    M llvm/lib/Target/AVR/AsmParser/AVRAsmParser.cpp
    M llvm/lib/Target/BPF/BTFDebug.h
    M llvm/lib/Target/MSP430/AsmParser/MSP430AsmParser.cpp
    M llvm/lib/Target/PowerPC/MCTargetDesc/PPCMCAsmInfo.h
    M llvm/lib/Target/X86/X86TargetTransformInfo.h
    M llvm/lib/Transforms/Utils/FixIrreducible.cpp
    M llvm/lib/Transforms/Utils/PredicateInfo.cpp
    M llvm/lib/Transforms/Utils/UnifyLoopExits.cpp

  Log Message:
  -----------
  [NFC] Add 'override' keyword where missing in include/ and lib/.

This fixes warnings raised by Clang's new -Wsuggest-override, in preparation for enabling that warning in the LLVM build. This patch also removes the virtual keyword where redundant, but only in places where doing so improves consistency within a given file. It also removes a couple unnecessary virtual destructor declarations in derived classes where the destructor inherited from the base class is already virtual.

Differential Revision: https://reviews.llvm.org/D83709




More information about the All-commits mailing list