[llvm-branch-commits] [llvm] [Dexter] Allow fetching "scopes" from the debugger (PR #202802)

[Orlando Cazalet-Hyams via llvm-branch-commits]

================
@@ -110,9 +110,6 @@ def get_active_wheres(where: Where, scope: Scope):
     # As we visit the script nodes in pre-order traversal, we can always assume that an expect's parent !where
     # has already been visited, and thus should have an entry in active_where_expects if it is active.
     def get_active_expects(expect: Expect, expected_value, scope: Scope):
----------------
OCHyams wrote:

for clarity, why do we remove the assert now? would it not be worth keeping and checking Value or ValueAll?

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