[PATCH] D111447: [Dexter] Add DexDeclareAddress command and address function

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

Orlando added a comment.

In D111447#3061202 <https://reviews.llvm.org/D111447#3061202>, @Orlando wrote:

> IMO "both variables available but different" should be treated as worse (higher penalty, lower desirability) than "one variable available". This fits the assumption that an incorrect value is more damaging to debugging experience than a missing one.

Then again, treating "one variable available" as "better" isn't good if the value that //is// available is incorrect, so it's a tricky situation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111447