[llvm-branch-commits] [lldb] [lldb] Add synthetic variable support to Get*VariableList. (PR #181501)

[Med Ismail Bennani via llvm-branch-commits]

================
@@ -704,7 +727,13 @@ static bool IsInRequestedScope(bool statics, bool arguments, bool locals,
   default:
     break;
   }
-  return false;
+
+  // The default for all other value types is !is_synthetic. At this point, if
+  // we didn't want extended variables we'd have exited by now anyway, so we
+  // must want them. Aside from the modifiers above that should apply equally to
+  // extended and normal variables, any other extended variable we should
+  // default to showing.
----------------
medismailben wrote:

```suggestion
  // The default for all other value types is !is_synthetic. At this point, if
  // we didn't want synthetic variables we'd have exited by now anyway, so we
  // must want them. Aside from the modifiers above that should apply equally to
  // synthetic and normal variables, any other extended variable we should
  // default to showing.
```

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