Fix an issue where LLDB would show the key and value of a single entry NSDictionary in the wrong order

Fixes rdar://26478641

llvm-svn: 273695

lldb/packages/Python/lldbsuite/test/lang/objc/single-entry-dictionary/Makefile

@@ -0,0 +1,7 @@
+LEVEL = ../../../make
+
+OBJC_SOURCES := main.m
+
+include $(LEVEL)/Makefile.rules
+
+LDFLAGS += -framework Foundation

lldb/packages/Python/lldbsuite/test/lang/objc/single-entry-dictionary/TestObjCSingleEntryDictionary.py

@@ -0,0 +1,68 @@
+"""Test that we properly vend children for a single entry NSDictionary"""
+
+from __future__ import print_function
+
+
+
+import unittest2
+import os, time
+import platform
+
+from distutils.version import StrictVersion
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
+
+class ObjCSingleEntryDictionaryTestCase(TestBase):
+
+    mydir = TestBase.compute_mydir(__file__)
+
+    def setUp(self):
+        # Call super's setUp().
+        TestBase.setUp(self)
+        # Find the line number to break inside main().
+        self.line = line_number('main.m', '// break here')
+
+    @skipUnlessDarwin
+    def test_single_entry_dict(self):
+        self.build()
+        exe = os.path.join(os.getcwd(), "a.out")
+        self.runCmd("file " + exe, CURRENT_EXECUTABLE_SET)
+
+        # Break inside the foo function which takes a bar_ptr argument.
+        lldbutil.run_break_set_by_file_and_line (self, "main.m", self.line, num_expected_locations=1, loc_exact=True)
+
+        self.runCmd("run", RUN_SUCCEEDED)
+
+        # The stop reason of the thread should be breakpoint.
+        self.expect("thread list", STOPPED_DUE_TO_BREAKPOINT,
+            substrs = ['stopped',
+                       'stop reason = breakpoint'])
+
+        # The breakpoint should have a hit count of 1.
+        self.expect("breakpoint list -f", BREAKPOINT_HIT_ONCE,
+            substrs = [' resolved, hit count = 1'])
+
+        d1 = self.frame().FindVariable("d1")
+        d1.SetPreferSyntheticValue(True)
+        d1.SetPreferDynamicValue(lldb.eDynamicCanRunTarget)
+        
+        self.assertTrue(d1.GetNumChildren() == 1, "dictionary has != 1 child elements")
+        pair = d1.GetChildAtIndex(0)
+        pair.SetPreferSyntheticValue(True)
+        pair.SetPreferDynamicValue(lldb.eDynamicCanRunTarget)
+        
+        self.assertTrue(pair.GetNumChildren() == 2, "pair has != 2 child elements")
+        
+        key = pair.GetChildMemberWithName("key")
+        value = pair.GetChildMemberWithName("value")
+        
+        key.SetPreferSyntheticValue(True)
+        key.SetPreferDynamicValue(lldb.eDynamicCanRunTarget)
+        value.SetPreferSyntheticValue(True)
+        value.SetPreferDynamicValue(lldb.eDynamicCanRunTarget)
+        
+        self.assertTrue(key.GetSummary() == '@"key"', "key doesn't contain key")
+        self.assertTrue(value.GetSummary() == '@"value"', "value doesn't contain value")

lldb/packages/Python/lldbsuite/test/lang/objc/single-entry-dictionary/main.m

@@ -0,0 +1,7 @@
+#import <Foundation/Foundation.h>
+
+int main() {
+  NSDictionary *d1 = @{@"key" : @"value"};
+  NSLog(@"%@\n", d1); // break here
+  return 0;
+}

lldb/source/Plugins/Language/ObjC/NSDictionary.cpp

@@ -593,10 +593,10 @@ lldb_private::formatters::NSDictionary1SyntheticFrontEnd::GetChildAtIndex (size_
     
     Error error;
     
-    lldb::addr_t key_at_idx = process_sp->ReadPointerFromMemory(key_ptr, error);
+    lldb::addr_t value_at_idx = process_sp->ReadPointerFromMemory(key_ptr, error);
     if (error.Fail())
         return nullptr;
-    lldb::addr_t value_at_idx = process_sp->ReadPointerFromMemory(value_ptr, error);
+    lldb::addr_t key_at_idx = process_sp->ReadPointerFromMemory(value_ptr, error);
     if (error.Fail())
         return nullptr;