[PATCH] D115449: [Dexter] Allow DexUnreachable to be specified in supplementary .dex files

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

Orlando accepted this revision.
Orlando added a comment.
This revision is now accepted and ready to land.

Please can you update Commands.md too? LGTM with nit, thanks.



================
Comment at: cross-project-tests/debuginfo-tests/dexter/dex/debugger/DebuggerControllers/ControllerHelpers.py:22-23
+def have_hit_line(watch, loc):
+  if watch.lineno == loc.lineno:
+    return True
+  elif hasattr(watch, '_on_line') and watch._on_line == loc.lineno:
----------------
Shouldn't this check come last? Otherwise IIUC this will return True when the positional line number of the command is hit, even when a line argument is specified.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115449/new/

https://reviews.llvm.org/D115449