Merge pull request #1844 from jimingham/no-vars-not-no-debug-info

Fix reporting the lack of global variables in "target var".

Author: jimingham

lldb/source/Commands/CommandObjectTarget.cpp

@@ -922,6 +922,7 @@ class CommandObjectTargetVariable : public CommandObjectParsed {
         CompileUnit *comp_unit = nullptr;
         if (frame) {
           SymbolContext sc = frame->GetSymbolContext(eSymbolContextCompUnit);
+          comp_unit = sc.comp_unit;
           if (sc.comp_unit) {
             const bool can_create = true;
             VariableListSP comp_unit_varlist_sp(

lldb/test/API/functionalities/target_var/no_vars/Makefile

@@ -0,0 +1,3 @@
+C_SOURCES := main.c
+
+include Makefile.rules

lldb/test/API/functionalities/target_var/no_vars/TestTargetVarNoVars.py

@@ -0,0 +1,21 @@
+"""
+Test that target var with no variables returns a correct error
+"""
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
+
+
+class TestTargetVarNoVars(TestBase):
+
+    mydir = TestBase.compute_mydir(__file__)
+
+    NO_DEBUG_INFO_TESTCASE = True
+
+    def test_target_var_no_vars(self):
+        self.build()
+        lldbutil.run_to_name_breakpoint(self, 'main')
+        self.expect("target variable", substrs=['no global variables in current compile unit', 'main.c'], error=True)
+

lldb/test/API/functionalities/target_var/no_vars/main.c

@@ -0,0 +1,5 @@
+int
+main(int argc, char **argv)
+{
+  return argc;
+}