[lldb] Detect not-captured variables in DIL's impl of GetValueForVariableExpressionPath

This requires changing the VisitIdentifier method. Like in the legacy
implementation, we can upstream these changes once we have a C++
mechanism for detecting the parent closure of a closure from its mangled
name.

Author: felipepiovezan

lldb/source/ValueObject/DILEval.cpp

@@ -161,6 +161,15 @@ Interpreter::Visit(const IdentifierNode *node) {
     identifier = LookupGlobalIdentifier(node->GetName(), m_exe_ctx_scope,
                                         m_target, use_dynamic);
   if (!identifier) {
+    // BEGIN SWIFT
+    // Implement LanguageCPlusPlus::GetParentNameIfClosure and upstream this.
+    // rdar://152321823
+    if (std::string message =
+            m_exe_ctx_scope->GetVariableNotCapturedDiagnostic(node->GetName());
+        !message.empty())
+      return llvm::make_error<DILDiagnosticError>(
+          m_expr, message, node->GetLocation(), node->GetName().size());
+    // END SWIFT
     std::string errMsg =
         llvm::formatv("use of undeclared identifier '{0}'", node->GetName());
     return llvm::make_error<DILDiagnosticError>(

lldb/test/API/lang/swift/closures_var_not_captured/TestSwiftClosureVarNotCaptured.py

@@ -54,8 +54,6 @@ def get_to_bkpt(self, bkpt_name):
     @swiftTest
     def test_simple_closure(self):
         self.build()
-        # rdar://158447239
-        self.runCmd('settings set target.experimental.use-DIL false')
         (target, process, thread) = self.get_to_bkpt("break_simple_closure")
         check_not_captured_error(self, thread.frames[0], "var_in_foo", "func_1(arg:)")
         check_not_captured_error(self, thread.frames[0], "arg", "func_1(arg:)")
@@ -64,8 +62,6 @@ def test_simple_closure(self):
     @swiftTest
     def test_nested_closure(self):
         self.build()
-        # rdar://158447239
-        self.runCmd('settings set target.experimental.use-DIL false')
         (target, process, thread) = self.get_to_bkpt("break_double_closure_1")
         check_not_captured_error(self, thread.frames[0], "var_in_foo", "func_2(arg:)")
         check_not_captured_error(self, thread.frames[0], "arg", "func_2(arg:)")
@@ -92,8 +88,6 @@ def test_nested_closure(self):
     @skipIf(oslist=["windows", "linux"])
     def test_async_closure(self):
         self.build()
-        # rdar://158447239
-        self.runCmd('settings set target.experimental.use-DIL false')
         (target, process, thread) = self.get_to_bkpt("break_async_closure_1")
         check_not_captured_error(self, thread.frames[0], "var_in_foo", "func_3(arg:)")
         check_not_captured_error(self, thread.frames[0], "arg", "func_3(arg:)")
@@ -115,8 +109,6 @@ def test_async_closure(self):
     @swiftTest
     def test_ctor_class_closure(self):
         self.build()
-        # rdar://158447239
-        self.runCmd('settings set target.experimental.use-DIL false')
         (target, process, thread) = self.get_to_bkpt("break_ctor_class")
         check_not_captured_error(
             self, thread.frames[0], "input", "MY_CLASS.init(input:)"
@@ -174,8 +166,6 @@ def test_ctor_class_closure(self):
     @swiftTest
     def test_ctor_struct_closure(self):
         self.build()
-        # rdar://158447239
-        self.runCmd('settings set target.experimental.use-DIL false')
         (target, process, thread) = self.get_to_bkpt("break_ctor_struct")
         check_not_captured_error(
             self, thread.frames[0], "input", "MY_STRUCT.init(input:)"
@@ -233,8 +223,6 @@ def test_ctor_struct_closure(self):
     @swiftTest
     def test_ctor_enum_closure(self):
         self.build()
-        # rdar://158447239
-        self.runCmd('settings set target.experimental.use-DIL false')
         (target, process, thread) = self.get_to_bkpt("break_ctor_enum")
         check_not_captured_error(
             self, thread.frames[0], "input", "MY_ENUM.init(input:)"