[Lldb-commits] [lldb] [lldb] Fix SBAPI breakage in SBFrame::GetValueForVariablePath (PR #218565)

[Ebuka Ezike via lldb-commits]

================
@@ -182,16 +182,19 @@ class LLDB_API SBFrame {
   // expression result and is not a constant object like
   // SBFrame::EvaluateExpression(...) returns, but a child object of the
   // variable value.
-  lldb::SBValue
-  GetValueForVariablePath(const char *var_expr_cstr,
-                          DynamicValueType use_dynamic,
-                          lldb::DILMode mode = lldb::eDILModeFull);
+  lldb::SBValue GetValueForVariablePath(const char *var_path,
+                                        DynamicValueType use_dynamic);
 
   /// The version that doesn't supply a 'use_dynamic' value will use the
   /// target's default.
-  lldb::SBValue
-  GetValueForVariablePath(const char *var_path,
-                          lldb::DILMode mode = lldb::eDILModeFull);
+  lldb::SBValue GetValueForVariablePath(const char *var_path);
+
+  lldb::SBValue GetValueForVariablePathWithMode(const char *var_path,
----------------
da-viper wrote:

We can have 
`GetValueForVariablePath(const char *, DynamicValueType, lldb::DILMode)` but the user will have to specify the dynamic value every time to specify the DILMode. as we cannot change the order of the parameters. 

https://github.com/llvm/llvm-project/pull/218565