[Lldb-commits] [PATCH] D106584: [lldb] Assert file cache and live memory are equal on debug builds

[Jason Molenda via Phabricator via lldb-commits]

jasonmolenda added a comment.

Looks good!  Maybe we should add a setting like target.file-cache-memory-reads-verify (a boolean) to enable/disable this check instead of keying off NDEBUG.  We can turn it on in the testsuite, and we can optionally have people with release installs of lldb enable it if we ever suspect that it might be an issue.  The settings are added with an entry in source/Target/TargetProperties.td and a getter/setter in TargetProperties.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106584