Skip to content

Commit

Permalink
Fix Clang-tidy modernize-use-override warnings in some files in sourc…
Browse files Browse the repository at this point in the history
…e/Plugins; other minor fixes.

Differential Revision: http://reviews.llvm.org/D13840

llvm-svn: 250721
  • Loading branch information
EugeneZelenko committed Oct 19, 2015
1 parent 755e502 commit 0af149a
Show file tree
Hide file tree
Showing 17 changed files with 314 additions and 295 deletions.
9 changes: 5 additions & 4 deletions lldb/source/Plugins/Process/elf-core/ProcessElfCore.h
@@ -1,4 +1,4 @@
//===-- ProcessElfCore.h ---------------------------------------*- C++ -*-===//
//===-- ProcessElfCore.h ----------------------------------------*- C++ -*-===//
//
// The LLVM Compiler Infrastructure
//
Expand All @@ -17,11 +17,13 @@
#ifndef liblldb_ProcessElfCore_h_
#define liblldb_ProcessElfCore_h_

// C Includes
// C++ Includes
#include <list>
#include <vector>

// Other libraries and framework includes
// Project includes
#include "lldb/Core/ConstString.h"
#include "lldb/Core/Error.h"
#include "lldb/Target/Process.h"
Expand Down Expand Up @@ -60,8 +62,7 @@ class ProcessElfCore : public lldb_private::Process
lldb_private::Listener &listener,
const lldb_private::FileSpec &core_file);

virtual
~ProcessElfCore();
~ProcessElfCore() override;

//------------------------------------------------------------------
// Check if a given Process
Expand Down Expand Up @@ -157,4 +158,4 @@ class ProcessElfCore : public lldb_private::Process
AddAddressRangeFromLoadSegment(const elf::ELFProgramHeader *header);
};

#endif // liblldb_ProcessElffCore_h_
#endif // liblldb_ProcessElfCore_h_
40 changes: 23 additions & 17 deletions lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_arm.h
@@ -1,15 +1,19 @@
//===-- RegisterContextCorePOSIX_arm.h -----------------------*- C++ -*-===//
//===-- RegisterContextCorePOSIX_arm.h --------------------------*- C++ -*-===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===---------------------------------------------------------------------===//
//===----------------------------------------------------------------------===//

#ifndef liblldb_RegisterContextCorePOSIX_arm_H_
#define liblldb_RegisterContextCorePOSIX_arm_H_
#ifndef liblldb_RegisterContextCorePOSIX_arm_h_
#define liblldb_RegisterContextCorePOSIX_arm_h_

// C Includes
// C++ Includes
// Other libraries and framework includes
// Project includes
#include "lldb/Core/DataBufferHeap.h"
#include "Plugins/Process/Utility/RegisterContextPOSIX_arm.h"

Expand All @@ -22,39 +26,41 @@ class RegisterContextCorePOSIX_arm :
const lldb_private::DataExtractor &gpregset,
const lldb_private::DataExtractor &fpregset);

~RegisterContextCorePOSIX_arm();
~RegisterContextCorePOSIX_arm() override;

virtual bool
ReadRegister(const lldb_private::RegisterInfo *reg_info, lldb_private::RegisterValue &value);
bool
ReadRegister(const lldb_private::RegisterInfo *reg_info,
lldb_private::RegisterValue &value) override;

virtual bool
WriteRegister(const lldb_private::RegisterInfo *reg_info, const lldb_private::RegisterValue &value);
bool
WriteRegister(const lldb_private::RegisterInfo *reg_info,
const lldb_private::RegisterValue &value) override;

bool
ReadAllRegisterValues(lldb::DataBufferSP &data_sp);
ReadAllRegisterValues(lldb::DataBufferSP &data_sp) override;

bool
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp);
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp) override;

bool
HardwareSingleStep(bool enable);
HardwareSingleStep(bool enable) override;

protected:
bool
ReadGPR();
ReadGPR() override;

bool
ReadFPR();
ReadFPR() override;

bool
WriteGPR();
WriteGPR() override;

bool
WriteFPR();
WriteFPR() override;

private:
lldb::DataBufferSP m_gpr_buffer;
lldb_private::DataExtractor m_gpr;
};

#endif // #ifndef liblldb_RegisterContextCorePOSIX_arm_H_
#endif // liblldb_RegisterContextCorePOSIX_arm_h_
@@ -1,15 +1,19 @@
//===-- RegisterContextCorePOSIX_arm64.h -----------------------*- C++ -*-===//
//===-- RegisterContextCorePOSIX_arm64.h ------------------------*- C++ -*-===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===---------------------------------------------------------------------===//
//===----------------------------------------------------------------------===//

#ifndef liblldb_RegisterContextCorePOSIX_arm64_H_
#define liblldb_RegisterContextCorePOSIX_arm64_H_
#ifndef liblldb_RegisterContextCorePOSIX_arm64_h_
#define liblldb_RegisterContextCorePOSIX_arm64_h_

// C Includes
// C++ Includes
// Other libraries and framework includes
// Project includes
#include "lldb/Core/DataBufferHeap.h"
#include "Plugins/Process/Utility/RegisterContextPOSIX_arm64.h"

Expand All @@ -22,39 +26,41 @@ class RegisterContextCorePOSIX_arm64 :
const lldb_private::DataExtractor &gpregset,
const lldb_private::DataExtractor &fpregset);

~RegisterContextCorePOSIX_arm64();
~RegisterContextCorePOSIX_arm64() override;

virtual bool
ReadRegister(const lldb_private::RegisterInfo *reg_info, lldb_private::RegisterValue &value);
bool
ReadRegister(const lldb_private::RegisterInfo *reg_info,
lldb_private::RegisterValue &value) override;

virtual bool
WriteRegister(const lldb_private::RegisterInfo *reg_info, const lldb_private::RegisterValue &value);
bool
WriteRegister(const lldb_private::RegisterInfo *reg_info,
const lldb_private::RegisterValue &value) override;

bool
ReadAllRegisterValues(lldb::DataBufferSP &data_sp);
ReadAllRegisterValues(lldb::DataBufferSP &data_sp) override;

bool
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp);
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp) override;

bool
HardwareSingleStep(bool enable);
HardwareSingleStep(bool enable) override;

protected:
bool
ReadGPR();
ReadGPR() override;

bool
ReadFPR();
ReadFPR() override;

bool
WriteGPR();
WriteGPR() override;

bool
WriteFPR();
WriteFPR() override;

private:
lldb::DataBufferSP m_gpr_buffer;
lldb_private::DataExtractor m_gpr;
};

#endif // #ifndef liblldb_RegisterContextCorePOSIX_arm64_H_
#endif // liblldb_RegisterContextCorePOSIX_arm64_h_
@@ -1,15 +1,19 @@
//===-- RegisterContextCorePOSIX_mips64.h ----------------------*- C++ -*-===//
//===-- RegisterContextCorePOSIX_mips64.h -----------------------*- C++ -*-===//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===---------------------------------------------------------------------===//
//===----------------------------------------------------------------------===//

#ifndef liblldb_RegisterContextCorePOSIX_mips64_H_
#define liblldb_RegisterContextCorePOSIX_mips64_H_
#ifndef liblldb_RegisterContextCorePOSIX_mips64_h_
#define liblldb_RegisterContextCorePOSIX_mips64_h_

// C Includes
// C++ Includes
// Other libraries and framework includes
// Project includes
#include "lldb/Core/DataBufferHeap.h"
#include "Plugins/Process/Utility/RegisterContextPOSIX_mips64.h"

Expand All @@ -22,39 +26,41 @@ class RegisterContextCorePOSIX_mips64 :
const lldb_private::DataExtractor &gpregset,
const lldb_private::DataExtractor &fpregset);

~RegisterContextCorePOSIX_mips64();
~RegisterContextCorePOSIX_mips64() override;

virtual bool
ReadRegister(const lldb_private::RegisterInfo *reg_info, lldb_private::RegisterValue &value);
bool
ReadRegister(const lldb_private::RegisterInfo *reg_info,
lldb_private::RegisterValue &value) override;

virtual bool
WriteRegister(const lldb_private::RegisterInfo *reg_info, const lldb_private::RegisterValue &value);
bool
WriteRegister(const lldb_private::RegisterInfo *reg_info,
const lldb_private::RegisterValue &value) override;

bool
ReadAllRegisterValues(lldb::DataBufferSP &data_sp);
ReadAllRegisterValues(lldb::DataBufferSP &data_sp) override;

bool
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp);
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp) override;

bool
HardwareSingleStep(bool enable);
HardwareSingleStep(bool enable) override;

protected:
bool
ReadGPR();
ReadGPR() override;

bool
ReadFPR();
ReadFPR() override;

bool
WriteGPR();
WriteGPR() override;

bool
WriteFPR();
WriteFPR() override;

private:
lldb::DataBufferSP m_gpr_buffer;
lldb_private::DataExtractor m_gpr;
};

#endif // #ifndef liblldb_RegisterContextCorePOSIX_mips64_H_
#endif // liblldb_RegisterContextCorePOSIX_mips64_h_
Expand Up @@ -5,11 +5,15 @@
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===---------------------------------------------------------------------===//
//===----------------------------------------------------------------------===//

#ifndef liblldb_RegisterContextCorePOSIX_powerpc_H_
#define liblldb_RegisterContextCorePOSIX_powerpc_H_
#ifndef liblldb_RegisterContextCorePOSIX_powerpc_h_
#define liblldb_RegisterContextCorePOSIX_powerpc_h_

// C Includes
// C++ Includes
// Other libraries and framework includes
// Project includes
#include "lldb/Core/DataBufferHeap.h"
#include "Plugins/Process/Utility/RegisterContextPOSIX_powerpc.h"

Expand All @@ -23,41 +27,43 @@ class RegisterContextCorePOSIX_powerpc :
const lldb_private::DataExtractor &fpregset,
const lldb_private::DataExtractor &vregset);

~RegisterContextCorePOSIX_powerpc();
~RegisterContextCorePOSIX_powerpc() override;

virtual bool
ReadRegister(const lldb_private::RegisterInfo *reg_info, lldb_private::RegisterValue &value);
bool
ReadRegister(const lldb_private::RegisterInfo *reg_info,
lldb_private::RegisterValue &value) override;

virtual bool
WriteRegister(const lldb_private::RegisterInfo *reg_info, const lldb_private::RegisterValue &value);
bool
WriteRegister(const lldb_private::RegisterInfo *reg_info,
const lldb_private::RegisterValue &value) override;

bool
ReadAllRegisterValues(lldb::DataBufferSP &data_sp);
ReadAllRegisterValues(lldb::DataBufferSP &data_sp) override;

bool
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp);
WriteAllRegisterValues(const lldb::DataBufferSP &data_sp) override;

bool
HardwareSingleStep(bool enable);
HardwareSingleStep(bool enable) override;

protected:
bool
ReadGPR();
ReadGPR() override;

bool
ReadFPR();
ReadFPR() override;

bool
ReadVMX();
ReadVMX() override;

bool
WriteGPR();
WriteGPR() override;

bool
WriteFPR();
WriteFPR() override;

bool
WriteVMX();
WriteVMX() override;

private:
lldb::DataBufferSP m_gpr_buffer;
Expand All @@ -68,4 +74,4 @@ class RegisterContextCorePOSIX_powerpc :
lldb_private::DataExtractor m_vec;
};

#endif // #ifndef liblldb_RegisterContextCorePOSIX_powerpc_H_
#endif // liblldb_RegisterContextCorePOSIX_powerpc_h_

0 comments on commit 0af149a

Please sign in to comment.