[Lldb-commits] [PATCH] D155905: lldb RFC: Exposing set/get address masks, Fix*Address methods in SBProcess

[David Spickett via Phabricator via lldb-commits]

DavidSpickett added a comment.

> I like it the above approach with more enums for the high and low code/data. Not sure if eTypeAny makes sense in the GetAddressMask(eTypeAny) scenario

It could be like the `FixAny` method, a fallback when you don't know either way. `eTypeAny` returns `FixAnyAddress(~0)` essentially.

Or it could or together all the masks but while that's good for curiosity's sake (what address bits are "safe"?) you would still need specific masks to construct an address for specific purposes.

It could fail, but then we're carrying around a Status object for one case, and that seems a shame. Unless we can think of other failure modes we haven't considered yet.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155905