llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-lldb Author: Felipe de Azevedo Piovezan (felipepiovezan) <details> <summary>Changes</summary> Based on testing on processors that use pointer metadata, and with all the work done to delay calls to FixDataAddress, this is no longer necessary. Note that, with debugserver in particular, this is an NFC change: the code path here is for frame zero, and debugserver will strip metadata when reading fp from frame zero anyway. --- Full diff: https://github.com/llvm/llvm-project/pull/159606.diff 1 Files Affected: - (modified) lldb/source/Target/RegisterContextUnwind.cpp (-3) ``````````diff diff --git a/lldb/source/Target/RegisterContextUnwind.cpp b/lldb/source/Target/RegisterContextUnwind.cpp index 3b018c09b8b72..197e1dd090bac 100644 --- a/lldb/source/Target/RegisterContextUnwind.cpp +++ b/lldb/source/Target/RegisterContextUnwind.cpp @@ -2228,9 +2228,6 @@ bool RegisterContextUnwind::ReadGPRValue(lldb::RegisterKind register_kind, if (generic_regnum == LLDB_REGNUM_GENERIC_PC || generic_regnum == LLDB_REGNUM_GENERIC_RA) value = abi_sp->FixCodeAddress(value); - if (generic_regnum == LLDB_REGNUM_GENERIC_SP || - generic_regnum == LLDB_REGNUM_GENERIC_FP) - value = abi_sp->FixDataAddress(value); } return true; } `````````` </details> https://github.com/llvm/llvm-project/pull/159606 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
