[Lldb-commits] [PATCH] D146320: [lldb] Test direct ivar access in objc++ (NFC)

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 17 11:45:29 PDT 2023


kastiglione created this revision.
kastiglione added a reviewer: aprantl.
Herald added a project: All.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

Add an Objective-C++ specific test for direct ivar access. This adds to the existing C++ and ObjC tests, and tests against regression for future refactoring.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D146320

Files:
  lldb/test/API/commands/frame/var/direct-ivar/objcpp/Makefile
  lldb/test/API/commands/frame/var/direct-ivar/objcpp/TestFrameVarDirectIvarObjCPlusPlus.py
  lldb/test/API/commands/frame/var/direct-ivar/objcpp/main.mm


Index: lldb/test/API/commands/frame/var/direct-ivar/objcpp/main.mm
===================================================================
--- /dev/null
+++ lldb/test/API/commands/frame/var/direct-ivar/objcpp/main.mm
@@ -0,0 +1,34 @@
+#include <objc/NSObject.h>
+
+struct Structure {
+  int m_field;
+  void fun() {
+    // check this
+  }
+};
+
+ at interface Classic : NSObject {
+ at public
+  int _ivar;
+}
+ at end
+
+ at implementation Classic
+- (void)fun {
+  // check self
+}
+ at end
+
+int main() {
+  Structure s;
+  s.m_field = 41;
+  s.fun();
+
+  Classic *c = [Classic new];
+  c->_ivar = 30;
+  [c fun];
+
+  Classic *self = c;
+  // check explicit self
+  (void)self;
+}
Index: lldb/test/API/commands/frame/var/direct-ivar/objcpp/TestFrameVarDirectIvarObjCPlusPlus.py
===================================================================
--- /dev/null
+++ lldb/test/API/commands/frame/var/direct-ivar/objcpp/TestFrameVarDirectIvarObjCPlusPlus.py
@@ -0,0 +1,24 @@
+import lldb
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test.decorators import *
+from lldbsuite.test import lldbutil
+
+
+class TestCase(TestBase):
+    @skipUnlessDarwin
+    def test_objc_self(self):
+        self.build()
+        lldbutil.run_to_source_breakpoint(self, "// check self", lldb.SBFileSpec("main.mm"))
+        self.expect("frame variable _ivar", startstr="(int) _ivar = 30")
+
+    @skipUnlessDarwin
+    def test_objc_explicit_self(self):
+        self.build()
+        lldbutil.run_to_source_breakpoint(self, "// check explicit self", lldb.SBFileSpec("main.mm"))
+        self.expect("frame variable _ivar", startstr="(int) _ivar = 30")
+
+    @skipUnlessDarwin
+    def test_cpp_this(self):
+        self.build()
+        lldbutil.run_to_source_breakpoint(self, "// check this", lldb.SBFileSpec("main.mm"))
+        self.expect("frame variable m_field", startstr="(int) m_field = 41")
Index: lldb/test/API/commands/frame/var/direct-ivar/objcpp/Makefile
===================================================================
--- /dev/null
+++ lldb/test/API/commands/frame/var/direct-ivar/objcpp/Makefile
@@ -0,0 +1,4 @@
+OBJCXX_SOURCES := main.mm
+CFLAGS_EXTRAS := -fblocks -fobjc-arc
+LD_EXTRAS := -lobjc
+include Makefile.rules


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D146320.506157.patch
Type: text/x-patch
Size: 2218 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20230317/2f7bf45f/attachment.bin>


More information about the lldb-commits mailing list