[Lldb-commits] [lldb] [LLDB] Add field member operators to DIL (PR #138093)
via lldb-commits
lldb-commits at lists.llvm.org
Thu May 1 00:10:31 PDT 2025
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 HEAD~1...HEAD lldb/test/API/commands/frame/var-dil/basics/MemberOf/TestFrameVarDILMemberOf.py lldb/test/API/commands/frame/var-dil/basics/MemberOfAnonymousMember/TestFrameVarDILMemberOfAnonymousMember.py lldb/test/API/commands/frame/var-dil/basics/MemberOfInheritance/TestFrameVarDILMemberOfInheritance.py
``````````
</details>
<details>
<summary>
View the diff from darker here.
</summary>
``````````diff
--- MemberOf/TestFrameVarDILMemberOf.py 2025-05-01 07:05:57.000000 +0000
+++ MemberOf/TestFrameVarDILMemberOf.py 2025-05-01 07:10:01.907042 +0000
@@ -9,39 +9,50 @@
import os
import shutil
import time
+
class TestFrameVarDILMemberOf(TestBase):
# If your test case doesn't stress debug info, then
# set this to true. That way it won't be run once for
# each debug info format.
NO_DEBUG_INFO_TESTCASE = True
def test_frame_var(self):
self.build()
- lldbutil.run_to_source_breakpoint(self, "Set a breakpoint here",
- lldb.SBFileSpec("main.cpp"))
+ lldbutil.run_to_source_breakpoint(
+ self, "Set a breakpoint here", lldb.SBFileSpec("main.cpp")
+ )
- self.expect("settings set target.experimental.use-DIL true",
- substrs=[""])
+ self.expect("settings set target.experimental.use-DIL true", substrs=[""])
self.expect_var_path("s.x", value="1")
self.expect_var_path("s.r", value="2")
self.expect_var_path("sr.x", value="1")
self.expect_var_path("sr.r", value="2")
self.expect_var_path("sp->x", value="1")
self.expect_var_path("sp->r", value="2")
- self.expect_var_path("sarr->x", value="5");
+ self.expect_var_path("sarr->x", value="5")
self.expect_var_path("sarr->r", value="2")
- self.expect("frame variable 'sp->foo'", error=True,
- substrs=["no member named 'foo' in 'Sx'"])
+ self.expect(
+ "frame variable 'sp->foo'",
+ error=True,
+ substrs=["no member named 'foo' in 'Sx'"],
+ )
- self.expect("frame variable 'sp.x'", error=True,
- substrs=["member reference type 'Sx *' is a "
- "pointer; did you mean to use '->'"])
- self.expect("frame variable 'sarr.x'", error=True,
- substrs=["no member named 'x' in 'Sx[2]'"])
+ self.expect(
+ "frame variable 'sp.x'",
+ error=True,
+ substrs=[
+ "member reference type 'Sx *' is a " "pointer; did you mean to use '->'"
+ ],
+ )
+ self.expect(
+ "frame variable 'sarr.x'",
+ error=True,
+ substrs=["no member named 'x' in 'Sx[2]'"],
+ )
# Test for record typedefs.
self.expect_var_path("sa.x", value="3")
self.expect_var_path("sa.y", value="'\\x04'")
--- MemberOfAnonymousMember/TestFrameVarDILMemberOfAnonymousMember.py 2025-05-01 07:05:57.000000 +0000
+++ MemberOfAnonymousMember/TestFrameVarDILMemberOfAnonymousMember.py 2025-05-01 07:10:01.933926 +0000
@@ -9,29 +9,31 @@
import os
import shutil
import time
+
class TestFrameVarDILMemberOfAnonymousMember(TestBase):
# If your test case doesn't stress debug info, then
# set this to true. That way it won't be run once for
# each debug info format.
NO_DEBUG_INFO_TESTCASE = True
def test_frame_var(self):
self.build()
- lldbutil.run_to_source_breakpoint(self, "Set a breakpoint here",
- lldb.SBFileSpec("main.cpp"))
+ lldbutil.run_to_source_breakpoint(
+ self, "Set a breakpoint here", lldb.SBFileSpec("main.cpp")
+ )
- self.expect("settings set target.experimental.use-DIL true",
- substrs=[""])
+ self.expect("settings set target.experimental.use-DIL true", substrs=[""])
self.expect_var_path("a.x", value="1")
self.expect_var_path("a.y", value="2")
- self.expect("frame variable 'b.x'", error=True,
- substrs=["no member named 'x' in 'B'"])
- #self.expect_var_path("b.y", value="0")
+ self.expect(
+ "frame variable 'b.x'", error=True, substrs=["no member named 'x' in 'B'"]
+ )
+ # self.expect_var_path("b.y", value="0")
self.expect_var_path("b.z", value="3")
self.expect_var_path("b.w", value="4")
self.expect_var_path("b.a.x", value="1")
self.expect_var_path("b.a.y", value="2")
@@ -41,22 +43,30 @@
self.expect_var_path("d.x", value="7")
self.expect_var_path("d.y", value="8")
self.expect_var_path("d.z", value="9")
self.expect_var_path("d.w", value="10")
- self.expect("frame variable 'e.x'", error=True,
- substrs=["no member named 'x' in 'E'"])
- self.expect("frame variable 'f.x'", error=True,
- substrs=["no member named 'x' in 'F'"])
+ self.expect(
+ "frame variable 'e.x'", error=True, substrs=["no member named 'x' in 'E'"]
+ )
+ self.expect(
+ "frame variable 'f.x'", error=True, substrs=["no member named 'x' in 'F'"]
+ )
self.expect_var_path("f.named_field.x", value="12")
self.expect_var_path("unnamed_derived.y", value="2")
self.expect_var_path("unnamed_derived.z", value="13")
- self.expect("frame variable 'derb.x'", error=True,
- substrs=["no member named 'x' in 'DerivedB'"])
- self.expect("frame variable 'derb.y'", error=True,
- substrs=["no member named 'y' in 'DerivedB'"])
+ self.expect(
+ "frame variable 'derb.x'",
+ error=True,
+ substrs=["no member named 'x' in 'DerivedB'"],
+ )
+ self.expect(
+ "frame variable 'derb.y'",
+ error=True,
+ substrs=["no member named 'y' in 'DerivedB'"],
+ )
self.expect_var_path("derb.w", value="14")
self.expect_var_path("derb.k", value="15")
self.expect_var_path("derb.a.x", value="1")
self.expect_var_path("derb.a.y", value="2")
--- MemberOfInheritance/TestFrameVarDILMemberOfInheritance.py 2025-05-01 07:05:57.000000 +0000
+++ MemberOfInheritance/TestFrameVarDILMemberOfInheritance.py 2025-05-01 07:10:01.955423 +0000
@@ -9,23 +9,24 @@
import os
import shutil
import time
+
class TestFrameVarDILMemberOfInheritance(TestBase):
# If your test case doesn't stress debug info, then
# set this to true. That way it won't be run once for
# each debug info format.
NO_DEBUG_INFO_TESTCASE = True
def test_frame_var(self):
self.build()
- lldbutil.run_to_source_breakpoint(self, "Set a breakpoint here",
- lldb.SBFileSpec("main.cpp"))
+ lldbutil.run_to_source_breakpoint(
+ self, "Set a breakpoint here", lldb.SBFileSpec("main.cpp")
+ )
- self.expect("settings set target.experimental.use-DIL true",
- substrs=[""])
+ self.expect("settings set target.experimental.use-DIL true", substrs=[""])
self.expect_var_path("a.a_", value="1")
self.expect_var_path("b.b_", value="2")
self.expect_var_path("c.a_", value="1")
self.expect_var_path("c.b_", value="2")
self.expect_var_path("c.c_", value="3")
``````````
</details>
https://github.com/llvm/llvm-project/pull/138093
More information about the lldb-commits
mailing list