diff options
author | Sean Callanan <scallanan@apple.com> | 2013-04-24 21:25:20 +0000 |
---|---|---|
committer | Sean Callanan <scallanan@apple.com> | 2013-04-24 21:25:20 +0000 |
commit | 2ad6691be440e88e3719c73ba49c99239ab1d230 (patch) | |
tree | 36360db7fd4f4dbc60a6493422be78e379c00ad7 | |
parent | c0e5b3ba1d993892e3386a9f67e133a80b97ccc8 (diff) | |
download | bcm5719-llvm-2ad6691be440e88e3719c73ba49c99239ab1d230.tar.gz bcm5719-llvm-2ad6691be440e88e3719c73ba49c99239ab1d230.zip |
Fixed IRForTarget to handle Objective-C messages
sent to "super".
<rdar://problem/13042260>
llvm-svn: 180217
-rw-r--r-- | lldb/source/Expression/IRForTarget.cpp | 10 | ||||
-rw-r--r-- | lldb/test/lang/objc/objc-super/Makefile | 6 | ||||
-rw-r--r-- | lldb/test/lang/objc/objc-super/TestObjCSuper.py | 76 | ||||
-rw-r--r-- | lldb/test/lang/objc/objc-super/class.m | 39 |
4 files changed, 131 insertions, 0 deletions
diff --git a/lldb/source/Expression/IRForTarget.cpp b/lldb/source/Expression/IRForTarget.cpp index b4d96552379..778c6c0b183 100644 --- a/lldb/source/Expression/IRForTarget.cpp +++ b/lldb/source/Expression/IRForTarget.cpp @@ -1869,6 +1869,16 @@ IRForTarget::ResolveExternals (Function &llvm_function) return false; } } + else if (global_name.find("OBJC_CLASSLIST_SUP_REFS_$") != global_name.npos) + { + if (!HandleObjCClass(global)) + { + if (m_error_stream) + m_error_stream->Printf("Error [IRForTarget]: Couldn't resolve the class for an Objective-C static method call\n"); + + return false; + } + } else if (DeclForGlobal(global)) { if (!MaybeHandleVariable (global)) diff --git a/lldb/test/lang/objc/objc-super/Makefile b/lldb/test/lang/objc/objc-super/Makefile new file mode 100644 index 00000000000..c2d07910a6e --- /dev/null +++ b/lldb/test/lang/objc/objc-super/Makefile @@ -0,0 +1,6 @@ +LEVEL = ../../../make + +OBJC_SOURCES := class.m +LDFLAGS = $(CFLAGS) -lobjc -framework Foundation + +include $(LEVEL)/Makefile.rules diff --git a/lldb/test/lang/objc/objc-super/TestObjCSuper.py b/lldb/test/lang/objc/objc-super/TestObjCSuper.py new file mode 100644 index 00000000000..746e29cf4b5 --- /dev/null +++ b/lldb/test/lang/objc/objc-super/TestObjCSuper.py @@ -0,0 +1,76 @@ +"""Test calling methods on super.""" + +import os, time +import unittest2 +import lldb +import lldbutil +from lldbtest import * + +class TestObjCSuperMethod(TestBase): + + mydir = os.path.join("lang", "objc", "objc-super") + + @unittest2.skipUnless(sys.platform.startswith("darwin"), "requires Darwin") + @expectedFailurei386 + @python_api_test + @dsym_test + def test_with_dsym_and_python_api(self): + """Test calling methods on super.""" + self.buildDsym() + self.objc_super() + + @unittest2.skipUnless(sys.platform.startswith("darwin"), "requires Darwin") + @expectedFailurei386 + @python_api_test + @dwarf_test + def test_with_dwarf_and_python_api(self): + """Test calling methods on super.""" + self.buildDwarf() + self.objc_super() + + def setUp(self): + # Call super's setUp(). + TestBase.setUp(self) + # Find the line numbers to break inside main(). + self.main_source = "class.m" + self.break_line = line_number(self.main_source, '// Set breakpoint here.') + + def objc_super(self): + """Test calling methods on super.""" + exe = os.path.join(os.getcwd(), "a.out") + + target = self.dbg.CreateTarget(exe) + self.assertTrue(target, VALID_TARGET) + + bpt = target.BreakpointCreateByLocation(self.main_source, self.break_line) + self.assertTrue(bpt, VALID_BREAKPOINT) + + # Now launch the process, and do not stop at entry point. + process = target.LaunchSimple (None, None, os.getcwd()) + + self.assertTrue(process, PROCESS_IS_VALID) + + # The stop reason of the thread should be breakpoint. + thread_list = lldbutil.get_threads_stopped_at_breakpoint (process, bpt) + + # Make sure we stopped at the first breakpoint. + self.assertTrue (len(thread_list) != 0, "No thread stopped at our breakpoint.") + self.assertTrue (len(thread_list) == 1, "More than one thread stopped at our breakpoint.") + + # Now make sure we can call a function in the class method we've stopped in. + frame = thread_list[0].GetFrameAtIndex(0) + self.assertTrue (frame, "Got a valid frame 0 frame.") + + cmd_value = frame.EvaluateExpression ("[self get]") + self.assertTrue (cmd_value.IsValid()) + self.assertTrue (cmd_value.GetValueAsUnsigned() == 2) + + cmd_value = frame.EvaluateExpression ("[super get]") + self.assertTrue (cmd_value.IsValid()) + self.assertTrue (cmd_value.GetValueAsUnsigned() == 1) + +if __name__ == '__main__': + import atexit + lldb.SBDebugger.Initialize() + atexit.register(lambda: lldb.SBDebugger.Terminate()) + unittest2.main() diff --git a/lldb/test/lang/objc/objc-super/class.m b/lldb/test/lang/objc/objc-super/class.m new file mode 100644 index 00000000000..b55b649aaae --- /dev/null +++ b/lldb/test/lang/objc/objc-super/class.m @@ -0,0 +1,39 @@ +#import <Foundation/Foundation.h> + +@interface Foo : NSObject { +} +-(int)get; +@end + +@implementation Foo +-(int)get +{ + return 1; +} +@end + +@interface Bar : Foo { +} +-(int)get; +@end + +@implementation Bar +-(int)get +{ + return 2; +} + +-(int)callme +{ + return [self get]; // Set breakpoint here. +} +@end + +int main() +{ + @autoreleasepool + { + Bar *bar = [Bar alloc]; + return [bar callme]; + } +} |