[Lldb-commits] [lldb] ee56cb9 - [lldb] remove a superfluous assignment statement (#152669)

[via lldb-commits]

Author: Matej Košík
Date: 2025-08-11T20:48:34-05:00
New Revision: ee56cb9c10bddf438c630353245abafdb32e6e9f

URL: https://github.com/llvm/llvm-project/commit/ee56cb9c10bddf438c630353245abafdb32e6e9f
DIFF: https://github.com/llvm/llvm-project/commit/ee56cb9c10bddf438c630353245abafdb32e6e9f.diff

LOG: [lldb] remove a superfluous assignment statement (#152669)

`cfa_reg_contents` is a local variable. Whatever value we assign there
right before the `return` statement will be lost anyway.

Co-authored-by: Matej Košík <matej.kosik at codasip.com>

Added: 
    

Modified: 
    lldb/source/Target/RegisterContextUnwind.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Target/RegisterContextUnwind.cpp b/lldb/source/Target/RegisterContextUnwind.cpp
index 9e9e2d86958f3..bcf1297f2114f 100644
--- a/lldb/source/Target/RegisterContextUnwind.cpp
+++ b/lldb/source/Target/RegisterContextUnwind.cpp
@@ -2026,7 +2026,6 @@ bool RegisterContextUnwind::ReadFrameAddress(
             "Got an invalid CFA register value - reg %s (%d), value 0x%" PRIx64,
             cfa_reg.GetName(), cfa_reg.GetAsKind(eRegisterKindLLDB),
             cfa_reg_contents);
-        cfa_reg_contents = LLDB_INVALID_ADDRESS;
         return false;
       }
       address = cfa_reg_contents + fa.GetOffset();