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
+  }
+};
+
+@interface Classic : NSObject {
+@public
+  int _ivar;
+}
+@end
+
+@implementation Classic
+- (void)fun {
+  // check self
+}
+@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


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
+  }
+};
+
+@interface Classic : NSObject {
+@public
+  int _ivar;
+}
+@end
+
+@implementation Classic
+- (void)fun {
+  // check self
+}
+@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
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to