[Lldb-commits] [lldb] [lldb][RISCV] Support optionally disabled FPR (PR #104547)
Alexey Merzlyakov via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 15 23:31:36 PDT 2024
https://github.com/AlexeyMerzlyakov updated https://github.com/llvm/llvm-project/pull/104547
>From f0dcd28876e3815454be3c9b2ae19cab26dede1e Mon Sep 17 00:00:00 2001
From: Alexey Merzlyakov <alexey.merzlyakov at samsung.com>
Date: Thu, 15 Aug 2024 14:29:49 +0300
Subject: [PATCH 1/2] Support optionally disabled FPR
---
.../RISCV/EmulateInstructionRISCV.cpp | 8 +-
.../NativeRegisterContextLinux_riscv64.cpp | 57 +++++---
.../NativeRegisterContextLinux_riscv64.h | 2 +
.../Utility/RegisterContextPOSIX_riscv64.cpp | 3 +-
.../Utility/RegisterInfoPOSIX_riscv64.cpp | 132 ++++++++++--------
.../Utility/RegisterInfoPOSIX_riscv64.h | 38 +++--
.../Process/Utility/RegisterInfos_riscv64.h | 6 +-
.../RegisterContextPOSIXCore_riscv64.cpp | 25 ++--
.../RegisterContextPOSIXCore_riscv64.h | 3 -
.../postmortem/elf-core/TestLinuxCore.py | 67 +++++++++
.../elf-core/linux-riscv64.no_fpr.core | Bin 0 -> 28672 bytes
.../elf-core/linux-riscv64.no_fpr.out | Bin 0 -> 3520 bytes
.../postmortem/elf-core/main_fpr.c | 17 +++
13 files changed, 251 insertions(+), 107 deletions(-)
create mode 100644 lldb/test/API/functionalities/postmortem/elf-core/linux-riscv64.no_fpr.core
create mode 100755 lldb/test/API/functionalities/postmortem/elf-core/linux-riscv64.no_fpr.out
create mode 100644 lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c
diff --git a/lldb/source/Plugins/Instruction/RISCV/EmulateInstructionRISCV.cpp b/lldb/source/Plugins/Instruction/RISCV/EmulateInstructionRISCV.cpp
index e8014b1eeb3789..badc7ba36f0118 100644
--- a/lldb/source/Plugins/Instruction/RISCV/EmulateInstructionRISCV.cpp
+++ b/lldb/source/Plugins/Instruction/RISCV/EmulateInstructionRISCV.cpp
@@ -1748,10 +1748,10 @@ EmulateInstructionRISCV::GetRegisterInfo(RegisterKind reg_kind,
}
}
- const RegisterInfo *array =
- RegisterInfoPOSIX_riscv64::GetRegisterInfoPtr(m_arch);
- const uint32_t length =
- RegisterInfoPOSIX_riscv64::GetRegisterInfoCount(m_arch);
+ RegisterInfoPOSIX_riscv64 reg_info(m_arch,
+ RegisterInfoPOSIX_riscv64::eRegsetMaskAll);
+ const RegisterInfo *array = reg_info.GetRegisterInfo();
+ const uint32_t length = reg_info.GetRegisterCount();
if (reg_index >= length || reg_kind != eRegisterKindLLDB)
return {};
diff --git a/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.cpp b/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.cpp
index 1d51726a86df16..aa8e6a0869aacb 100644
--- a/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.cpp
+++ b/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.cpp
@@ -23,12 +23,11 @@
// System includes - They have to be included after framework includes because
// they define some macros which collide with variable names in other modules
+#include <sys/ptrace.h>
#include <sys/uio.h>
// NT_PRSTATUS and NT_FPREGSET definition
#include <elf.h>
-#define REG_CONTEXT_SIZE (GetGPRSize() + GetFPRSize())
-
using namespace lldb;
using namespace lldb_private;
using namespace lldb_private::process_linux;
@@ -38,7 +37,21 @@ NativeRegisterContextLinux::CreateHostNativeRegisterContextLinux(
const ArchSpec &target_arch, NativeThreadLinux &native_thread) {
switch (target_arch.GetMachine()) {
case llvm::Triple::riscv64: {
- Flags opt_regsets;
+ Flags opt_regsets(RegisterInfoPOSIX_riscv64::eRegsetMaskDefault);
+
+ RegisterInfoPOSIX_riscv64::FPR fpr;
+ struct iovec ioVec;
+ ioVec.iov_base = &fpr;
+ ioVec.iov_len = sizeof(fpr);
+ unsigned int regset = NT_FPREGSET;
+
+ if (NativeProcessLinux::PtraceWrapper(PTRACE_GETREGSET,
+ native_thread.GetID(), ®set,
+ &ioVec, sizeof(fpr))
+ .Success()) {
+ opt_regsets.Set(RegisterInfoPOSIX_riscv64::eRegsetMaskFP);
+ }
+
auto register_info_up =
std::make_unique<RegisterInfoPOSIX_riscv64>(target_arch, opt_regsets);
return std::make_unique<NativeRegisterContextLinux_riscv64>(
@@ -193,20 +206,23 @@ Status NativeRegisterContextLinux_riscv64::ReadAllRegisterValues(
lldb::WritableDataBufferSP &data_sp) {
Status error;
- data_sp.reset(new DataBufferHeap(REG_CONTEXT_SIZE, 0));
+ data_sp.reset(new DataBufferHeap(GetRegContextSize(), 0));
error = ReadGPR();
if (error.Fail())
return error;
- error = ReadFPR();
- if (error.Fail())
- return error;
+ if (GetRegisterInfo().IsFPPresent()) {
+ error = ReadFPR();
+ if (error.Fail())
+ return error;
+ }
uint8_t *dst = const_cast<uint8_t *>(data_sp->GetBytes());
::memcpy(dst, GetGPRBuffer(), GetGPRSize());
dst += GetGPRSize();
- ::memcpy(dst, GetFPRBuffer(), GetFPRSize());
+ if (GetRegisterInfo().IsFPPresent())
+ ::memcpy(dst, GetFPRBuffer(), GetFPRSize());
return error;
}
@@ -222,11 +238,11 @@ Status NativeRegisterContextLinux_riscv64::WriteAllRegisterValues(
return error;
}
- if (data_sp->GetByteSize() != REG_CONTEXT_SIZE) {
+ if (data_sp->GetByteSize() != GetRegContextSize()) {
error.SetErrorStringWithFormat(
"NativeRegisterContextLinux_riscv64::%s data_sp contained mismatched "
"data size, expected %" PRIu64 ", actual %" PRIu64,
- __FUNCTION__, REG_CONTEXT_SIZE, data_sp->GetByteSize());
+ __FUNCTION__, GetRegContextSize(), data_sp->GetByteSize());
return error;
}
@@ -245,23 +261,32 @@ Status NativeRegisterContextLinux_riscv64::WriteAllRegisterValues(
return error;
src += GetRegisterInfoInterface().GetGPRSize();
- ::memcpy(GetFPRBuffer(), src, GetFPRSize());
- error = WriteFPR();
- if (error.Fail())
- return error;
+ if (GetRegisterInfo().IsFPPresent()) {
+ ::memcpy(GetFPRBuffer(), src, GetFPRSize());
+
+ error = WriteFPR();
+ if (error.Fail())
+ return error;
+ }
return error;
}
+size_t NativeRegisterContextLinux_riscv64::GetRegContextSize() {
+ size_t size = GetGPRSize();
+ if (GetRegisterInfo().IsFPPresent())
+ size += GetFPRSize();
+ return size;
+}
+
bool NativeRegisterContextLinux_riscv64::IsGPR(unsigned reg) const {
return GetRegisterInfo().GetRegisterSetFromRegisterIndex(reg) ==
RegisterInfoPOSIX_riscv64::GPRegSet;
}
bool NativeRegisterContextLinux_riscv64::IsFPR(unsigned reg) const {
- return GetRegisterInfo().GetRegisterSetFromRegisterIndex(reg) ==
- RegisterInfoPOSIX_riscv64::FPRegSet;
+ return GetRegisterInfo().IsFPReg(reg);
}
Status NativeRegisterContextLinux_riscv64::ReadGPR() {
diff --git a/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.h b/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.h
index 41b4e2573add9c..d5cc50131cdc30 100644
--- a/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.h
+++ b/lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_riscv64.h
@@ -75,6 +75,8 @@ class NativeRegisterContextLinux_riscv64 : public NativeRegisterContextLinux {
RegisterInfoPOSIX_riscv64::FPR m_fpr;
+ size_t GetRegContextSize();
+
bool IsGPR(unsigned reg) const;
bool IsFPR(unsigned reg) const;
diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_riscv64.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_riscv64.cpp
index 035ce00e116269..bbcfb9eae10034 100644
--- a/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_riscv64.cpp
+++ b/lldb/source/Plugins/Process/Utility/RegisterContextPOSIX_riscv64.cpp
@@ -77,6 +77,5 @@ bool RegisterContextPOSIX_riscv64::IsGPR(unsigned int reg) {
}
bool RegisterContextPOSIX_riscv64::IsFPR(unsigned int reg) {
- return m_register_info_up->GetRegisterSetFromRegisterIndex(reg) ==
- RegisterInfoPOSIX_riscv64::FPRegSet;
+ return m_register_info_up->IsFPReg(reg);
}
diff --git a/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.cpp b/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.cpp
index 3819401c543b1b..4a3737795848e9 100644
--- a/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.cpp
+++ b/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.cpp
@@ -18,42 +18,15 @@
#define GPR_OFFSET(idx) ((idx)*8 + 0)
#define FPR_OFFSET(idx) ((idx)*8 + sizeof(RegisterInfoPOSIX_riscv64::GPR))
-#define REG_CONTEXT_SIZE \
- (sizeof(RegisterInfoPOSIX_riscv64::GPR) + \
- sizeof(RegisterInfoPOSIX_riscv64::FPR))
-
#define DECLARE_REGISTER_INFOS_RISCV64_STRUCT
#include "RegisterInfos_riscv64.h"
#undef DECLARE_REGISTER_INFOS_RISCV64_STRUCT
-const lldb_private::RegisterInfo *RegisterInfoPOSIX_riscv64::GetRegisterInfoPtr(
- const lldb_private::ArchSpec &target_arch) {
- switch (target_arch.GetMachine()) {
- case llvm::Triple::riscv64:
- return g_register_infos_riscv64_le;
- default:
- assert(false && "Unhandled target architecture.");
- return nullptr;
- }
-}
-
-uint32_t RegisterInfoPOSIX_riscv64::GetRegisterInfoCount(
- const lldb_private::ArchSpec &target_arch) {
- switch (target_arch.GetMachine()) {
- case llvm::Triple::riscv64:
- return static_cast<uint32_t>(sizeof(g_register_infos_riscv64_le) /
- sizeof(g_register_infos_riscv64_le[0]));
- default:
- assert(false && "Unhandled target architecture.");
- return 0;
- }
-}
-
// Number of register sets provided by this context.
enum {
k_num_gpr_registers = gpr_last_riscv - gpr_first_riscv + 1,
k_num_fpr_registers = fpr_last_riscv - fpr_first_riscv + 1,
- k_num_register_sets = 2
+ k_num_register_sets_default = 1
};
// RISC-V64 general purpose registers.
@@ -73,38 +46,69 @@ static_assert(((sizeof g_gpr_regnums_riscv64 /
1) == k_num_gpr_registers,
"g_gpr_regnums_riscv64 has wrong number of register infos");
-// RISC-V64 floating point registers.
-static const uint32_t g_fpr_regnums_riscv64[] = {
- fpr_f0_riscv, fpr_f1_riscv, fpr_f2_riscv, fpr_f3_riscv,
- fpr_f4_riscv, fpr_f5_riscv, fpr_f6_riscv, fpr_f7_riscv,
- fpr_f8_riscv, fpr_f9_riscv, fpr_f10_riscv, fpr_f11_riscv,
- fpr_f12_riscv, fpr_f13_riscv, fpr_f14_riscv, fpr_f15_riscv,
- fpr_f16_riscv, fpr_f17_riscv, fpr_f18_riscv, fpr_f19_riscv,
- fpr_f20_riscv, fpr_f21_riscv, fpr_f22_riscv, fpr_f23_riscv,
- fpr_f24_riscv, fpr_f25_riscv, fpr_f26_riscv, fpr_f27_riscv,
- fpr_f28_riscv, fpr_f29_riscv, fpr_f30_riscv, fpr_f31_riscv,
- fpr_fcsr_riscv, LLDB_INVALID_REGNUM};
-
-static_assert(((sizeof g_fpr_regnums_riscv64 /
- sizeof g_fpr_regnums_riscv64[0]) -
- 1) == k_num_fpr_registers,
- "g_fpr_regnums_riscv64 has wrong number of register infos");
-
// Register sets for RISC-V64.
-static const lldb_private::RegisterSet g_reg_sets_riscv64[k_num_register_sets] =
- {{"General Purpose Registers", "gpr", k_num_gpr_registers,
- g_gpr_regnums_riscv64},
- {"Floating Point Registers", "fpr", k_num_fpr_registers,
- g_fpr_regnums_riscv64}};
+static const lldb_private::RegisterSet g_reg_set_gpr_riscv64 = {
+ "General Purpose Registers", "gpr", k_num_gpr_registers,
+ g_gpr_regnums_riscv64};
+static const lldb_private::RegisterSet g_reg_set_fpr_riscv64 = {
+ "Floating Point Registers", "fpr", k_num_fpr_registers, nullptr};
RegisterInfoPOSIX_riscv64::RegisterInfoPOSIX_riscv64(
- const lldb_private::ArchSpec &target_arch, lldb_private::Flags flags)
+ const lldb_private::ArchSpec &target_arch, lldb_private::Flags opt_regsets)
: lldb_private::RegisterInfoAndSetInterface(target_arch),
- m_register_info_p(GetRegisterInfoPtr(target_arch)),
- m_register_info_count(GetRegisterInfoCount(target_arch)) {}
+ m_opt_regsets(opt_regsets) {
+ switch (target_arch.GetMachine()) {
+ case llvm::Triple::riscv64: {
+ // By-default considering RISC-V has only GPR.
+ // Other register sets could be enabled optionally by opt_regsets.
+ AddRegSetGP();
+
+ if (m_opt_regsets.AnySet(eRegsetMaskFP))
+ AddRegSetFP();
+
+ break;
+ }
+ default:
+ assert(false && "Unhandled target architecture.");
+ }
+}
+
+void RegisterInfoPOSIX_riscv64::AddRegSetGP() {
+ m_register_infos.resize(k_num_gpr_registers);
+ memcpy(&m_register_infos[0], g_register_infos_riscv64_gpr,
+ sizeof(g_register_infos_riscv64_gpr));
+ m_register_sets.push_back(g_reg_set_gpr_riscv64);
+
+ m_per_regset_regnum_range[GPRegSet] =
+ std::make_pair(gpr_first_riscv, m_register_infos.size());
+}
+
+void RegisterInfoPOSIX_riscv64::AddRegSetFP() {
+ const uint32_t register_info_count = m_register_infos.size();
+ const uint32_t register_set_count = m_register_sets.size();
+
+ // Filling m_register_infos.
+ // For FPR case we do not need to correct register offsets and kinds
+ // while for other further cases (like VPR), register offset/kind
+ // should be started counting from the last one in previously added
+ // regset. This is needed for the case e.g. when architecture has GPR + VPR
+ // sets only.
+ m_register_infos.resize(register_info_count + k_num_fpr_registers);
+ memcpy(&m_register_infos[register_info_count], g_register_infos_riscv64_fpr,
+ sizeof(g_register_infos_riscv64_fpr));
+
+ // Filling m_register_sets with enabled register set
+ for (uint32_t i = 0; i < k_num_fpr_registers; i++)
+ m_fp_regnum_collection.push_back(register_info_count + i);
+ m_register_sets.push_back(g_reg_set_fpr_riscv64);
+ m_register_sets.back().registers = m_fp_regnum_collection.data();
+
+ m_per_regset_regnum_range[register_set_count] =
+ std::make_pair(register_info_count, m_register_infos.size());
+}
uint32_t RegisterInfoPOSIX_riscv64::GetRegisterCount() const {
- return m_register_info_count;
+ return m_register_infos.size();
}
size_t RegisterInfoPOSIX_riscv64::GetGPRSize() const {
@@ -117,26 +121,30 @@ size_t RegisterInfoPOSIX_riscv64::GetFPRSize() const {
const lldb_private::RegisterInfo *
RegisterInfoPOSIX_riscv64::GetRegisterInfo() const {
- return m_register_info_p;
+ return m_register_infos.data();
}
size_t RegisterInfoPOSIX_riscv64::GetRegisterSetCount() const {
- return k_num_register_sets;
+ return m_register_sets.size();
}
size_t RegisterInfoPOSIX_riscv64::GetRegisterSetFromRegisterIndex(
uint32_t reg_index) const {
- // coverity[unsigned_compare]
- if (reg_index >= gpr_first_riscv && reg_index <= gpr_last_riscv)
- return GPRegSet;
- if (reg_index >= fpr_first_riscv && reg_index <= fpr_last_riscv)
- return FPRegSet;
+ for (const auto ®set_range : m_per_regset_regnum_range) {
+ if (reg_index >= regset_range.second.first &&
+ reg_index < regset_range.second.second)
+ return regset_range.first;
+ }
return LLDB_INVALID_REGNUM;
}
+bool RegisterInfoPOSIX_riscv64::IsFPReg(unsigned reg) const {
+ return llvm::is_contained(m_fp_regnum_collection, reg);
+}
+
const lldb_private::RegisterSet *
RegisterInfoPOSIX_riscv64::GetRegisterSet(size_t set_index) const {
if (set_index < GetRegisterSetCount())
- return &g_reg_sets_riscv64[set_index];
+ return &m_register_sets[set_index];
return nullptr;
}
diff --git a/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.h b/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.h
index 4bf4bede013284..f8e22c7df3c88f 100644
--- a/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.h
+++ b/lldb/source/Plugins/Process/Utility/RegisterInfoPOSIX_riscv64.h
@@ -11,19 +11,21 @@
#include "RegisterInfoAndSetInterface.h"
#include "lldb/Target/RegisterContext.h"
+#include "lldb/Utility/Flags.h"
#include "lldb/lldb-private.h"
#include <map>
class RegisterInfoPOSIX_riscv64
: public lldb_private::RegisterInfoAndSetInterface {
public:
- static const lldb_private::RegisterInfo *
- GetRegisterInfoPtr(const lldb_private::ArchSpec &target_arch);
- static uint32_t
- GetRegisterInfoCount(const lldb_private::ArchSpec &target_arch);
+ enum { GPRegSet = 0 };
-public:
- enum { GPRegSet = 0, FPRegSet };
+ // RISC-V64 register set mask value
+ enum {
+ eRegsetMaskDefault = 0,
+ eRegsetMaskFP = 1,
+ eRegsetMaskAll = -1,
+ };
struct GPR {
// note: gpr[0] is pc, not x0
@@ -41,7 +43,11 @@ class RegisterInfoPOSIX_riscv64
};
RegisterInfoPOSIX_riscv64(const lldb_private::ArchSpec &target_arch,
- lldb_private::Flags flags);
+ lldb_private::Flags opt_regsets);
+
+ void AddRegSetGP();
+
+ void AddRegSetFP();
size_t GetGPRSize() const override;
@@ -58,9 +64,23 @@ class RegisterInfoPOSIX_riscv64
size_t GetRegisterSetFromRegisterIndex(uint32_t reg_index) const override;
+ bool IsFPPresent() const { return m_opt_regsets.AnySet(eRegsetMaskFP); }
+
+ bool IsFPReg(unsigned reg) const;
+
private:
- const lldb_private::RegisterInfo *m_register_info_p;
- uint32_t m_register_info_count;
+ std::vector<lldb_private::RegisterInfo> m_register_infos;
+
+ std::vector<lldb_private::RegisterSet> m_register_sets;
+
+ // Contains pair of [start, end] register numbers of a register set with start
+ // and end included.
+ std::map<uint32_t, std::pair<uint32_t, uint32_t>> m_per_regset_regnum_range;
+
+ // Register collections to be stored as reference for m_register_sets items
+ std::vector<uint32_t> m_fp_regnum_collection;
+
+ lldb_private::Flags m_opt_regsets;
};
#endif
diff --git a/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h b/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h
index 720d900c7b97e7..be337feabc6440 100644
--- a/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h
+++ b/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h
@@ -79,7 +79,7 @@ using namespace riscv_dwarf;
// clang-format on
-static lldb_private::RegisterInfo g_register_infos_riscv64_le[] = {
+static lldb_private::RegisterInfo g_register_infos_riscv64_gpr[] = {
// DEFINE_GPR64(name, GENERIC KIND)
DEFINE_GPR64(pc, LLDB_REGNUM_GENERIC_PC),
DEFINE_GPR64_ALT(ra, x1, LLDB_REGNUM_GENERIC_RA),
@@ -114,7 +114,9 @@ static lldb_private::RegisterInfo g_register_infos_riscv64_le[] = {
DEFINE_GPR64_ALT(t5, x30, LLDB_INVALID_REGNUM),
DEFINE_GPR64_ALT(t6, x31, LLDB_INVALID_REGNUM),
DEFINE_GPR64_ALT(zero, x0, LLDB_INVALID_REGNUM),
+};
+static lldb_private::RegisterInfo g_register_infos_riscv64_fpr[] = {
DEFINE_FPR64_ALT(ft0, f0, LLDB_INVALID_REGNUM),
DEFINE_FPR64_ALT(ft1, f1, LLDB_INVALID_REGNUM),
DEFINE_FPR64_ALT(ft2, f2, LLDB_INVALID_REGNUM),
@@ -148,7 +150,9 @@ static lldb_private::RegisterInfo g_register_infos_riscv64_le[] = {
DEFINE_FPR64_ALT(ft10, f30, LLDB_INVALID_REGNUM),
DEFINE_FPR64_ALT(ft11, f31, LLDB_INVALID_REGNUM),
DEFINE_FPR_ALT(fcsr, nullptr, 4, LLDB_INVALID_REGNUM),
+};
+static lldb_private::RegisterInfo g_register_infos_riscv64_vpr[] = {
DEFINE_VPR(v0, LLDB_INVALID_REGNUM),
DEFINE_VPR(v1, LLDB_INVALID_REGNUM),
DEFINE_VPR(v2, LLDB_INVALID_REGNUM),
diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.cpp b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.cpp
index 5ba18cdb9889a5..3dca4b16099056 100644
--- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.cpp
+++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.cpp
@@ -16,9 +16,17 @@ std::unique_ptr<RegisterContextCorePOSIX_riscv64>
RegisterContextCorePOSIX_riscv64::Create(Thread &thread, const ArchSpec &arch,
const DataExtractor &gpregset,
llvm::ArrayRef<CoreNote> notes) {
+ Flags opt_regsets = RegisterInfoPOSIX_riscv64::eRegsetMaskDefault;
+
+ DataExtractor fpregset = getRegset(notes, arch.GetTriple(), FPR_Desc);
+ if (fpregset.GetByteSize() >= sizeof(uint64_t)) {
+ opt_regsets.Set(RegisterInfoPOSIX_riscv64::eRegsetMaskFP);
+ }
+
return std::unique_ptr<RegisterContextCorePOSIX_riscv64>(
new RegisterContextCorePOSIX_riscv64(
- thread, std::make_unique<RegisterInfoPOSIX_riscv64>(arch, Flags()),
+ thread,
+ std::make_unique<RegisterInfoPOSIX_riscv64>(arch, opt_regsets),
gpregset, notes));
}
@@ -27,17 +35,14 @@ RegisterContextCorePOSIX_riscv64::RegisterContextCorePOSIX_riscv64(
const DataExtractor &gpregset, llvm::ArrayRef<CoreNote> notes)
: RegisterContextPOSIX_riscv64(thread, std::move(register_info)) {
- m_gpr_buffer = std::make_shared<DataBufferHeap>(gpregset.GetDataStart(),
- gpregset.GetByteSize());
- m_gpr.SetData(m_gpr_buffer);
+ m_gpr.SetData(std::make_shared<DataBufferHeap>(gpregset.GetDataStart(),
+ gpregset.GetByteSize()));
m_gpr.SetByteOrder(gpregset.GetByteOrder());
- ArchSpec arch = m_register_info_up->GetTargetArchitecture();
- DataExtractor fpregset = getRegset(notes, arch.GetTriple(), FPR_Desc);
- m_fpr_buffer = std::make_shared<DataBufferHeap>(fpregset.GetDataStart(),
- fpregset.GetByteSize());
- m_fpr.SetData(m_fpr_buffer);
- m_fpr.SetByteOrder(fpregset.GetByteOrder());
+ if (m_register_info_up->IsFPPresent()) {
+ ArchSpec arch = m_register_info_up->GetTargetArchitecture();
+ m_fpr = getRegset(notes, arch.GetTriple(), FPR_Desc);
+ }
}
RegisterContextCorePOSIX_riscv64::~RegisterContextCorePOSIX_riscv64() = default;
diff --git a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.h b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.h
index 3cf9531df2c1d2..a9a59844635749 100644
--- a/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.h
+++ b/lldb/source/Plugins/Process/elf-core/RegisterContextPOSIXCore_riscv64.h
@@ -50,9 +50,6 @@ class RegisterContextCorePOSIX_riscv64 : public RegisterContextPOSIX_riscv64 {
bool WriteFPR() override;
private:
- lldb::DataBufferSP m_gpr_buffer;
- lldb::DataBufferSP m_fpr_buffer;
-
lldb_private::DataExtractor m_gpr;
lldb_private::DataExtractor m_fpr;
};
diff --git a/lldb/test/API/functionalities/postmortem/elf-core/TestLinuxCore.py b/lldb/test/API/functionalities/postmortem/elf-core/TestLinuxCore.py
index 0b9d17bc9f45e6..60c47b094cf588 100644
--- a/lldb/test/API/functionalities/postmortem/elf-core/TestLinuxCore.py
+++ b/lldb/test/API/functionalities/postmortem/elf-core/TestLinuxCore.py
@@ -22,6 +22,7 @@ class LinuxCoreTestCase(TestBase):
_s390x_pid = 1045
_ppc64le_pid = 28147
_riscv64_pid = 89328
+ _riscv64_no_fpr_pid = 97
_aarch64_regions = 4
_i386_regions = 4
@@ -65,6 +66,17 @@ def test_riscv64(self):
"""Test that lldb can read the process information from an riscv64 linux core file."""
self.do_test("linux-riscv64", self._riscv64_pid, self._riscv64_regions, "a.out")
+ @skipIfLLVMTargetMissing("RISCV")
+ def test_riscv64_no_fpr(self):
+ """Test that lldb can read the process information from an riscv64 linux core file
+ made for a RV64IMAC target, having no FP-registers."""
+ self.do_test(
+ "linux-riscv64.no_fpr",
+ self._riscv64_no_fpr_pid,
+ self._riscv64_regions,
+ "a.out",
+ )
+
@skipIfLLVMTargetMissing("X86")
def test_same_pid_running(self):
"""Test that we read the information from the core correctly even if we have a running
@@ -760,6 +772,61 @@ def test_riscv64_regs(self):
self.expect("register read --all")
+ @skipIfLLVMTargetMissing("RISCV")
+ def test_riscv64_no_fpr_regs(self):
+ # check registers using 64 bit RISC-V core file containing GP-registers only
+ target = self.dbg.CreateTarget(None)
+ self.assertTrue(target, VALID_TARGET)
+ process = target.LoadCore("linux-riscv64.no_fpr.core")
+
+ values = {}
+ values["pc"] = "0x0000000000010164"
+ values["ra"] = "0x0000000000010194"
+ values["sp"] = "0x00fffffff4d5fcc0"
+ values["gp"] = "0x0000000000157678"
+ values["tp"] = "0x00ffffff99c43400"
+ values["t0"] = "0x00ffffff99c6b260"
+ values["t1"] = "0x00ffffff99b7bd54"
+ values["t2"] = "0x0000000003f0b27f"
+ values["fp"] = "0x00fffffff4d5fcf0"
+ values["s1"] = "0x0000000000000003"
+ values["a0"] = "0x0"
+ values["a1"] = "0x0000000000010144"
+ values["a2"] = "0x0000000000176460"
+ values["a3"] = "0x000000000015ee38"
+ values["a4"] = "0x00000000423c0000"
+ values["a5"] = "0x0"
+ values["a6"] = "0x0"
+ values["a7"] = "0x00000000000000dd"
+ values["s2"] = "0x0"
+ values["s3"] = "0x000000000014ddf8"
+ values["s4"] = "0x000000000003651c"
+ values["s5"] = "0x00fffffffccd8d28"
+ values["s6"] = "0x000000000014ddf8"
+ values["s7"] = "0x00ffffff99c69d48"
+ values["s8"] = "0x00ffffff99c6a008"
+ values["s9"] = "0x0"
+ values["s10"] = "0x0"
+ values["s11"] = "0x0"
+ values["t3"] = "0x00ffffff99c42000"
+ values["t4"] = "0x00ffffff99af8e20"
+ values["t5"] = "0x0000000000000005"
+ values["t6"] = "0x44760bdd8d5f6381"
+ values["zero"] = "0x0"
+
+ for regname, value in values.items():
+ self.expect(
+ "register read {}".format(regname),
+ substrs=["{} = {}".format(regname, value)],
+ )
+
+ # Check that LLDB does not try to read other registers from core file
+ self.expect(
+ "register read --all",
+ matching=False,
+ substrs=["registers were unavailable"],
+ )
+
def test_get_core_file_api(self):
"""
Test SBProcess::GetCoreFile() API can successfully get the core file.
diff --git a/lldb/test/API/functionalities/postmortem/elf-core/linux-riscv64.no_fpr.core b/lldb/test/API/functionalities/postmortem/elf-core/linux-riscv64.no_fpr.core
new file mode 100644
index 0000000000000000000000000000000000000000..63265a5db123d9ebc699a927d7477a264b558e91
GIT binary patch
literal 28672
zcmeHQeQ*=U72lJvBuoK=3Ni&!5dwDjkR?OF0i5^)AxI3kHeb_*2>B#iDz at a2WP>xb
z7}VJAOeheg*r84#Wd<{8n^4>t(@u#YA;7d at l1|$|r;tJ&aN0C{G{uG#P~W at TCyQ&C
zGG*H4k6k#uk9}|V?Qh>6d(7V4MrX+aog^_TgV-l*Kw_+bDpF_x%VTL=9>nRYRHqc%
zgJh7RLl(`IHZrK<E)7sUr)Z@#N&15GKT0GL)Cgl%&Js=WQ~I;>`Zf3NXF!)y?sx8Y
zf4`Uhu&?zHEXG^1Xvy+bK!@-qkfidY_5kxVTtrFV_!x02-TmC1a_E6O<yk*eWiOxC
z7dSn9Q;mIn{V3I7=Ft~%ENc!NQ|PioT?*ZJ;EEm%u5f!rP(762`MkX~UW+2<UE=Mr
zc||E^N%m}#Q<>{$zo;<dd}_<tTTb4 at I7q+nB{d$?Zk1}>2SlqHzx6w>DREOyZj#+w
z&JL-cPll*(feU*pURvf<^kvE~W5xo`sD!XjQd_7lnqpzg at u0?<qPi~@V{3$Kk!w*o
z#YVXy-B-*1xiiMJ8#>3lr<f*Cb9~)olg#PE8hMIFo~n_jY2 at h|*`|?mG%_v!=_Aw%
zWsM at sE4Z3@xgX_lfwCg`Jp}6Vcs-68i*j!PYE-0wAJYJ<aD%!0eZCfG&dBTw#Be%P
zBM;Na>Pi|=j^S*1LS3_8=JH>;JYcDE82?PToF$7E(3~anhrr at vXBo*F#W}|xs`06L
ztu`ip1p)#Afq+0jARrJB2nYlO0s;YnfIvVXAP^7;2m}NI0s(=5KtLcM5D*9m1Ox&C
z0fB%(Kp-Fx5C{ka1Ofs9fq+0jARrJB2nYl|w+L+1od4I~<@|ppKd_1?N7eKH1#r}M
zIG(Al#WS-XY7e~f5m#zPPA5;1(wSS0i6T;aMCn|w4r;N#6Z}B0mfCp-MCsfv`ry^V
z{j_u*)L|itwpdCpM7_vg=L#)-NU>s!C^u8in&TN^><-ToC(kEydm1~&M?N?<v`31=
zKp!b%%!mSs*=gL$x#VAt-Cw at d?jQHUbBVn_tC=unPjPVGsN*}c_E-48<*r4~(wUVy
z=2m!4dnCM#upwPlyv^~$R}N6Wkk9Kkx&60*j$l_)Ubs*z&<*Vw2b^%nHv>?g&J|%C
z2$VH&q5ikLecnC8WBI5*q0uw9yWSfKxx-O+Bv2LhMeD*r(#h2!d9Az37mWsLe5B-J
zsM70cAYIZh?2C{N?Bv=yk}9j~B7S#3_Q-x8s}2O~)=$T7ZSdEjzQ^I7X>&XRK|;~p
z at kiB3Fmw3l<?c&E at hyjOaMAk%h>;HJ>NfVKV;vU66AR`5>Tv#DOO~^FtxxPpoBHJS
z4g0^|mcC+Y_Q<~+IEnc{xl)3$fy{JjX3=?a>Wl2?y_{>e6CpI;#^;yPU>~0k%FlOs
zJ<Z2fbb><X+cQ=1p3ldVJ-pzd^k0etgmR0^ynA+q%Xz$BwfgV<N at MkNAA;twI}`li
zx_=iImlx<6pbECY)6seTGiBvW+8{||hw4y=<2nk#Q=X^Tk(uI)c3x<Y at I<?5a6Ofx
z15p%#!2bn-xM{yBqV!xi;=yx<518qYOa7oEJ<auO)m%#li#N}1xv{nNy+k73wD%it
zz0+P~?zgb9O-A;WkA^d=?o82--}xjf<FGl?$!4|19&6FX@=Ur|cN5ch8zlX{CQ1M5
zF$pd8dkm~+MCDCgW at mGMp?Q at _*V*4Ibv7IIUGMFj8*l2{)z{j2>bNA|>SMEK`Hq?m
zhBXIT)3SOl_1)H4r1;O<Chi|4wO{zb8l6{i7D}5N<E9KF#;#~4p{C2o8so?N6Q_%0
z)^F-OZqyBje?ouKWnO1A4E|`ltH*NfraG338^?DKPBV{ovMv*AoV6o<;(EezDv{7<
zxkBdk7AB7}fiFazXJ8Zer%C#}CN}Y>Y05|}r@=1!M0L*)WB=7I)hnx7PPYu1;Zl2C
z-M!sq=}jce#;OL{aE+;aM<Am(zHT&=CV#A8_f+vqe^jtDJJ}3V6;sfAyqW0%LH*-K
zDVE(V>21fAK4T9WS<m)2Zf;i4i=QU+Ec%^0w$;x*wR3JuIBw|QHKk2O at j$D5t<OX_
zO0HBS+b?t)jZ*vh)rUr=SuR49fb^6dmJe}Mez$pDCUacIvBzNSnU%;{dZ2FNp4B}M
z_;0?lqor~-%-EFHmWsHcZ`XqF%Aa0L&RA~ivoK>9Z*R72ioep at cA@?JrbCh|&_#1`
z>kwg7*i9Ly&d9Nqi;uJdvax;HE6x3xETCZ23Ud<caIN_GO+?B9d8nVfcZKDBptj(O
z*y~LLxpvT?%$Vgj7;8^i&rbiRy3v?x((`Jj9oOAkrp;;FY?-~XB4bnBY at B_zv$<C{
z(Vix~7$0|G`vu8~*@n4>QOEY~Euoy_^ZlgU)W<L<%*Ku<Vp+yv?MGr+u3=}6{GMk@
z^CI$)hxRkVWlUGxv at IR8S(?<)>ge&W?Wa<OP^DzgF>ptO7iofBu7lhcajToa+3@
zPW at xekU7neNsv8xn2T0j>t*nG?Q&tym-_x;wm{xA_ZCX=&i+%d_Ih<{?&twK%$_&e
z{swEXib3{V?bH2h_Lwt`I$XI6+HSzCp6sURUrx>?+&#{B7}Pb<JBLQv3gdwL$D^{L
zpYipVmvV>s_KmGudT@=7f3wSy5BD77woPxrs$;3EJ=W1Q472JiAZ698d&c6XMg_gi
zjmX<G5J}|e`JfPY#^FFju4mHMfE|>P7OtN)Gmu;BaM$G4+T9-HWaOR+$Q7X)hss65
zs;bf#lzm{0)&4**TI;Ec_!P0Grmh-{nMq0?;wmnlW6dsGRPMCZJ7(B&?YT3i+a1##
zk34L3%(U5Ub}OnJxsFGz*}nC)zA$W9gHca4ygg})_#@G9)KkH1Rlz!&-xKjOo3|kt
zX{b?YG_2Ot`@)exD0qk9_A*;A6!j^Vo|=HnY(BrcGVH1Gx&2;nB#GIgzV%Ts^F%$&
zrr=@oM5EzAMO_s3#5OrpgAoBh_gUb+t32P){*LycMDuw2wB4h96zyAazu5<SLh`45
zE75|x_;WSG=MHRx{ApiH^fWib{?LZ{$Nd_X at o-^m<WKu#qO>nYTP%FrtP1cu2`+f2
zIT=6gYl)8JmNb6qhiER;;~tr0+BXv={Iv1&W?3OHu*FQqf4ApdvM1b0b^(LW1QZ{w
zBcgUbVDcmUXdhe(zGy=6-|e{>Oq4?Hko+_-ss8lbO4Lh+oKpX!_H%!Fu1QWXH+)~y
zp+;mu?oZ=C$Dc=uYWpLZC at vxJt)+N~(*I{vl7dqXXHwe)9O~>zjmj+HFA~qu=*d5o
zQtYXwE!_X#<!`Ew-~|E#0fB%(Kp-Fx5C{ka1Ofs9fq+0jARrJB2nYlO0s;YnfIvVX
zAP^7;2m}NI0s(=5KtLcM5D*9m1Ox&C0fB%(Kp-Fx5C{ka1OopX1dbM2Z>itUp>JT(
zx2rz8|F??@xcx8qcYW@(d%dW%Mg3M)=2z7>DCxYdtobg_S}r$m`aGxYobKWDRZb6b
zdYIE5PO1M1#E9=NDM)<YzQ07%19aV-;<k+M<!5It`_t}UZ8}@SKN-f#7dlHy@^dNz
z!5nzE!{-TCH`IpVRg$&&5qJ^AQ#~yjZ78?1(v|b`l`k@}g-aJZ^K;lTXW8QX`l!R6
y%a)fr%ksmaP!t*~Sy7VjV5No27Ut(fpwT%wb&)XVk|Ho{X~}%0P4-#edHN?P8#2xS
literal 0
HcmV?d00001
diff --git a/lldb/test/API/functionalities/postmortem/elf-core/linux-riscv64.no_fpr.out b/lldb/test/API/functionalities/postmortem/elf-core/linux-riscv64.no_fpr.out
new file mode 100755
index 0000000000000000000000000000000000000000..908d61bb54caaac5764fa2e60ad14bdcd9ed2c30
GIT binary patch
literal 3520
zcmbtWU2GIp6h3!mx4UKgv!xi at 0O<lnieYDeTFQ^0?b1>j4I;$^cyO5Ao!t)1?kuyj
zO=(EP0`Vul5ECVe7>&lC#0L|7 at I@t>$b%0?UwAe#G0_Bwydl<eX6{{fr&}@cCik9m
z&-u<d_vhT7=O>@qFEIw9Nbo06?<$v#am6?79z(h@*a-n_qtFd9KqY{(s}cwL$cV2m
zA`uNX=^8ZvdG1e84CWS!MK#9ZF+NT~2R#@FN?0n-{OaV}JoZ1eJe(VA4O-wlVd~(s
zu&w*<fnyh+{Q0dPzkE3#n9slc^`{5UoNl|b at fP%L+VlQj*NW#qd~w~TNACQ6e#eD-
z<3)MP_v<cy%iz6=<V<Rd=Nhq_c7E>SL+d?!;GVqw!o9^e?xR^x+aDW$gD82Oa#7cc
zuU(v4!zORudgm^KUruE5LG(&f`Yr>o2kQ at 5zsiX5%L|8ron>ror{u0lA0iJeEiG*$
zlT24~;sxoYK&B%M&t1!0?~sz2E^u*Us6K)?#g9ADjGIOMal{RIxLDoaaTgaOe^dMR
z?T!r{o}QUh=2F8-I+-4hCsXm%_N}qhh>}#2F%qTHsa>%leZHdGMoBL_nz<FmhtaKV
zRC9B{wjvvd2JN}g5hGnmsikx!scOXKh+8O?YG$gE1e%$bJYgCWOz;d at bbHI2-%%6<
z=7Iwe?5rmYF|90vc~r2 at HgP`Od6VeMQEK-?H&N-532lJ at C3vV{S-?V_6+R4iL0y9n
z2#6IzeJ*?HsC0^Yo4+8%r9<efT{x4836r9oupobe&8n<s17n?HobH~YoLzC<GT{~-
zx%M<FU?U)hcW+>iMRsk7O!rEC7%K>JmnY&9xf>!I!qdID;yrA0S8qffSUb?eR)?Yw
z<6<f7iGDPQpo_?seiD!Mo$Y6cZX1M1<nf4%n;uda#%X~GKg71U;PxO;gjLId3;j5Y
z%UFXDhDT{ejNR?(Btpj?D>4I&_DEs_Qn$C=*h32j?UGoK-e5##?H$0dT?H&ilzOt;
zBV81w!3Hg&8nj`b7H!cIgJ&(P1O?O59J&-@`%*g|kjb%lNy{30&C2KqvPM;lS1WpM
zZ#tRUg?T?rYCAW(*G=w{mW#!UpjI8tc7Q_ZHk34 at tQIP^l7qx7N)j4M7tAx-DQhl4
zM=L>RNJ*uY)UfKFn9-5AX_Ra83DcY_#VfXTQqMWq$!8OeUUd at Vho%#SS~=$!R#`I*
z$Iz>ZidA(=mhI at Jgl-n%Im_1FRsHuy#vO_A^ln7TnBHP5n&mQFZ;21Ujh6CYqj{Z|
zq4kX}L#U2vWMs^yRz(K>xcQVa;AXcdvU at u$SIE$@Oi1|}fZI!c1tFz|^$};+ySJCG
z8(oBi_g^DBP#z`!ef&Y>fu>zKzJAeW`AL+oMnC at h>(~l-+W#oMf<8Z|SCF4uLH^<j
z^2Wrb>uhlTffqbosH$2tt*mCM70W0)x;==JnThcu<7#GdYWkq6Vm~u8qfQ>2aQSSt
z%FmOkHpBG1=4hbmWxUvx>Y0+GWsy3zM`s0Bwj5n4mTOA3W|;Z7kq5=m=N(XN%XOgW
zvueS{LkUXGD$)CeDtfpS&2elaTf;L4N?y;_imGO7<>LLWQ7%|@RW at tubE1dm7yWrg
zI1s^-7-wVSlK$_q7Cq2u5m%@<$KroA=!<}v&{XI-qYM)#QJnV)o_32(==*VmnEr<Z
z74gNraGl$We~iHU<L~j=i~HjSw|`X_a_YYU_H+BB4=4VeyM1!*)*Umcp2kXlU>~gh
zA71JIEASdJ!l#fC{fRG#e_Vbn1 at _CxUZM{xmYW`_w|?wd)c9otzRm4Fu3LJHANvZ5
z{W3zn$v5hXXI5|h*w;}*GXFeB`IYZaa$zs%yDQj##O)XAkv&G(HBym#jnE;U&_}rv
se>Bj8RpbwTo7_S#2VW{K at yIz2^yi<5E5PKlm3;?)l0FkaPW|@(0Q3BfZ~y=R
literal 0
HcmV?d00001
diff --git a/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c b/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c
new file mode 100644
index 00000000000000..db6c5ebd1c168b
--- /dev/null
+++ b/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c
@@ -0,0 +1,17 @@
+static void bar(float *boom)
+{
+ float F = 98.0;
+ *boom = 47.0; // Frame bar
+}
+
+static void foo(float *boom, void (*boomer)(float *))
+{
+ float F = 102.0;
+ boomer(boom); // Frame foo
+}
+
+void _start(void)
+{
+ float F = 95.0;
+ foo(0, bar); // Frame _start
+}
>From 1a8f739db3cde8dae7e06da3a41429ab00c7abd2 Mon Sep 17 00:00:00 2001
From: Alexey Merzlyakov <alexey.merzlyakov at samsung.com>
Date: Fri, 16 Aug 2024 09:30:28 +0300
Subject: [PATCH 2/2] Fix formatter issues
---
.../Process/Utility/RegisterInfos_riscv64.h | 48 +++++++------------
.../postmortem/elf-core/main_fpr.c | 21 ++++----
2 files changed, 25 insertions(+), 44 deletions(-)
diff --git a/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h b/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h
index be337feabc6440..628ed3770bdc05 100644
--- a/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h
+++ b/lldb/source/Plugins/Process/Utility/RegisterInfos_riscv64.h
@@ -153,38 +153,22 @@ static lldb_private::RegisterInfo g_register_infos_riscv64_fpr[] = {
};
static lldb_private::RegisterInfo g_register_infos_riscv64_vpr[] = {
- DEFINE_VPR(v0, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v1, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v2, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v3, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v4, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v5, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v6, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v7, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v8, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v9, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v10, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v11, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v12, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v13, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v14, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v15, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v16, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v17, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v18, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v19, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v20, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v21, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v22, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v23, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v24, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v25, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v26, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v27, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v28, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v29, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v30, LLDB_INVALID_REGNUM),
- DEFINE_VPR(v31, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v0, LLDB_INVALID_REGNUM), DEFINE_VPR(v1, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v2, LLDB_INVALID_REGNUM), DEFINE_VPR(v3, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v4, LLDB_INVALID_REGNUM), DEFINE_VPR(v5, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v6, LLDB_INVALID_REGNUM), DEFINE_VPR(v7, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v8, LLDB_INVALID_REGNUM), DEFINE_VPR(v9, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v10, LLDB_INVALID_REGNUM), DEFINE_VPR(v11, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v12, LLDB_INVALID_REGNUM), DEFINE_VPR(v13, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v14, LLDB_INVALID_REGNUM), DEFINE_VPR(v15, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v16, LLDB_INVALID_REGNUM), DEFINE_VPR(v17, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v18, LLDB_INVALID_REGNUM), DEFINE_VPR(v19, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v20, LLDB_INVALID_REGNUM), DEFINE_VPR(v21, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v22, LLDB_INVALID_REGNUM), DEFINE_VPR(v23, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v24, LLDB_INVALID_REGNUM), DEFINE_VPR(v25, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v26, LLDB_INVALID_REGNUM), DEFINE_VPR(v27, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v28, LLDB_INVALID_REGNUM), DEFINE_VPR(v29, LLDB_INVALID_REGNUM),
+ DEFINE_VPR(v30, LLDB_INVALID_REGNUM), DEFINE_VPR(v31, LLDB_INVALID_REGNUM),
};
#endif // DECLARE_REGISTER_INFOS_RISCV64_STRUCT
diff --git a/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c b/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c
index db6c5ebd1c168b..bcfe6d27359caa 100644
--- a/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c
+++ b/lldb/test/API/functionalities/postmortem/elf-core/main_fpr.c
@@ -1,17 +1,14 @@
-static void bar(float *boom)
-{
- float F = 98.0;
- *boom = 47.0; // Frame bar
+static void bar(float *boom) {
+ float F = 98.0;
+ *boom = 47.0; // Frame bar
}
-static void foo(float *boom, void (*boomer)(float *))
-{
- float F = 102.0;
- boomer(boom); // Frame foo
+static void foo(float *boom, void (*boomer)(float *)) {
+ float F = 102.0;
+ boomer(boom); // Frame foo
}
-void _start(void)
-{
- float F = 95.0;
- foo(0, bar); // Frame _start
+void _start(void) {
+ float F = 95.0;
+ foo(0, bar); // Frame _start
}
More information about the lldb-commits
mailing list