[Lldb-commits] [PATCH] D83847: [lldb][NFC] Add 'override' where missing in source/ and tools/

Logan Smith via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jul 14 22:13:17 PDT 2020


logan-5 created this revision.
logan-5 added a reviewer: clayborg.
logan-5 added a project: LLDB.
Herald added a subscriber: lldb-commits.

These were found by Clang's new `-Wsuggest-override`.

This patch doesn't touch any code in unittests/, since much of it intentionally doesn't use `override` to avoid massive warning spam from `-Winconsistent-missing-override` due to the use of `MOCK_***` macros.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D83847

Files:
  lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp
  lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm.h
  lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_arm64.h
  lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_i386.h
  lldb/source/Plugins/Process/MacOSX-Kernel/RegisterContextKDP_x86_64.h
  lldb/source/Plugins/Process/MacOSX-Kernel/ThreadKDP.h
  lldb/source/Plugins/Process/Utility/RegisterContextMach_arm.h
  lldb/source/Plugins/Process/Utility/RegisterContextMach_i386.h
  lldb/source/Plugins/Process/Utility/RegisterContextMach_x86_64.h
  lldb/source/Plugins/SymbolVendor/MacOSX/SymbolVendorMacOSX.h
  lldb/source/Symbol/FuncUnwinders.cpp
  lldb/tools/debugserver/source/MacOSX/i386/DNBArchImplI386.h
  lldb/tools/debugserver/source/MacOSX/x86_64/DNBArchImplX86_64.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D83847.278068.patch
Type: text/x-patch
Size: 21682 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200715/bced8c60/attachment-0001.bin>


More information about the lldb-commits mailing list