================
@@ -36,14 +37,16 @@ VariablesRequestHandler::Run(const VariablesArguments 
&arguments) const {
     int64_t start_idx = 0;
     int64_t num_children = 0;
 
-    if (var_ref == VARREF_REGS) {
+    std::optional<ScopeData> scope_data = dap.variables.GetScopeKind(var_ref);
+    if (scope_data.has_value() && scope_data->kind == eScopeKind::Registers) {
+
----------------
ashgti wrote:

Can we update `GetTopLevelScope` to return the `optional<ScopeData>` instead of 
having two ways of looking up this data?

That should make this a bit easier to follow and I think removes the need for 
`GetScopeKind`.

https://github.com/llvm/llvm-project/pull/124232
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to