[Lldb-commits] [lldb] [lldb] Remove redundant calls to std::unique_ptr<T>::get (NFC) (PR #139428)
Kazu Hirata via lldb-commits
lldb-commits at lists.llvm.org
Sat May 10 18:11:18 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/139428
None
>From 4001252c4cd0cc8fedfb1773ef3a53befe2774f6 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Tue, 25 Mar 2025 00:24:37 -0700
Subject: [PATCH] [lldb] Remove redundant calls to std::unique_ptr<T>::get
(NFC)
---
lldb/source/Target/RegisterContextUnwind.cpp | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/lldb/source/Target/RegisterContextUnwind.cpp b/lldb/source/Target/RegisterContextUnwind.cpp
index 4c760b84e45a5..f56dda187e12a 100644
--- a/lldb/source/Target/RegisterContextUnwind.cpp
+++ b/lldb/source/Target/RegisterContextUnwind.cpp
@@ -279,7 +279,7 @@ void RegisterContextUnwind::InitializeZerothFrame() {
call_site_unwind_plan = func_unwinders_sp->GetUnwindPlanAtCallSite(
process->GetTarget(), m_thread);
- if (call_site_unwind_plan.get() != nullptr) {
+ if (call_site_unwind_plan != nullptr) {
m_fallback_unwind_plan_sp = call_site_unwind_plan;
if (TryFallbackUnwindPlan())
cfa_status = true;
@@ -1722,10 +1722,10 @@ RegisterContextUnwind::SavedLocationForRegister(
// tricky frame and our usual techniques can continue to be used.
bool RegisterContextUnwind::TryFallbackUnwindPlan() {
- if (m_fallback_unwind_plan_sp.get() == nullptr)
+ if (m_fallback_unwind_plan_sp == nullptr)
return false;
- if (m_full_unwind_plan_sp.get() == nullptr)
+ if (m_full_unwind_plan_sp == nullptr)
return false;
if (m_full_unwind_plan_sp.get() == m_fallback_unwind_plan_sp.get() ||
@@ -1773,7 +1773,7 @@ bool RegisterContextUnwind::TryFallbackUnwindPlan() {
// fallback UnwindPlan. We checked if m_fallback_unwind_plan_sp was nullptr
// at the top -- the only way it became nullptr since then is via
// SavedLocationForRegister().
- if (m_fallback_unwind_plan_sp.get() == nullptr)
+ if (m_fallback_unwind_plan_sp == nullptr)
return true;
// Switch the full UnwindPlan to be the fallback UnwindPlan. If we decide
@@ -1862,10 +1862,10 @@ bool RegisterContextUnwind::TryFallbackUnwindPlan() {
}
bool RegisterContextUnwind::ForceSwitchToFallbackUnwindPlan() {
- if (m_fallback_unwind_plan_sp.get() == nullptr)
+ if (m_fallback_unwind_plan_sp == nullptr)
return false;
- if (m_full_unwind_plan_sp.get() == nullptr)
+ if (m_full_unwind_plan_sp == nullptr)
return false;
if (m_full_unwind_plan_sp.get() == m_fallback_unwind_plan_sp.get() ||
More information about the lldb-commits
mailing list