[Lldb-commits] [PATCH] D46083: Move the persistent variable counter into Target

[Jim Ingham via Phabricator via lldb-commits]

jingham requested changes to this revision.
jingham added a comment.
This revision now requires changes to proceed.

Except for the inline comment, this is okay.  If we thought hard we might find another likely reason why we'd want to use two result prefixes besides a language that has an error return side-channel.  But I can't off the top of my head, so is_error is fine for now.


https://reviews.llvm.org/D46083