[Lldb-commits] [lldb] Reapply "Fix error in unrecognized register name handling for "SBFram…e.register"" (#88468)" (PR #88535)

via lldb-commits lldb-commits at lists.llvm.org
Fri Apr 12 10:09:32 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {darker}-->


:warning: Python code formatter, darker found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
darker --check --diff -r e0a628715a8464e220c8ba9e9aaaf2561139198a...4a335c835239edc1989a1ddd7875a6e6edd2d143 lldb/test/API/python_api/frame/TestFrames.py
``````````

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- TestFrames.py	2024-04-12 17:00:07.000000 +0000
+++ TestFrames.py	2024-04-12 17:09:04.374356 +0000
@@ -71,12 +71,11 @@
                 # but they should be valid.  Uses get_GPRs() from the lldbutil
                 # module.
                 gpr_reg_set = lldbutil.get_GPRs(frame)
                 pc_value = gpr_reg_set.GetChildMemberWithName("pc")
                 self.assertTrue(pc_value, "We should have a valid PC.")
-                               
-    
+
                 # Make sure on arm targets we dont mismatch PC value on the basis of thumb bit.
                 # Frame PC will not have thumb bit set in case of a thumb
                 # instruction as PC.
                 pc_value_int = int(pc_value.GetValue(), 0)
                 if self.getArchitecture() in ["arm", "armv7", "armv7k"]:
@@ -100,12 +99,14 @@
                 for reg_set in frame.registers:
                     for reg in reg_set:
                         flattened_regs.add(reg.name)
                 for reg in frame.register:
                     register_regs.add(reg.name)
-                self.assertEqual(register_regs, flattened_regs, "register matches registers")
-                        
+                self.assertEqual(
+                    register_regs, flattened_regs, "register matches registers"
+                )
+
             print("---", file=session)
             process.Continue()
 
         # At this point, the inferior process should have exited.
         self.assertEqual(process.GetState(), lldb.eStateExited, PROCESS_EXITED)

``````````

</details>


https://github.com/llvm/llvm-project/pull/88535


More information about the lldb-commits mailing list