[Lldb-commits] [PATCH] D91422: [lldb] [Process/FreeBSDRemote] Check for regset support early [WIP]

[Muhammad Omair Javaid via Phabricator via lldb-commits]

omjavaid added a comment.

In D91422#2396697 <https://reviews.llvm.org/D91422#2396697>, @labath wrote:

> My guess is that this is because of the extra debug registers that you're exposing. You'll notice that Get(User)RegisterSetCount returns just a number, and this means that it is not really possible to selectively disable register sets -- all you can do is disable the ones that come last.
>
> We ran into a similar problem on arm with @omjavaid.
>
> For the time being, we may be able to work around this by making the debug registers come earlier in the list (after ensuring they work on linux). Or we could hide them on BSDs as well...

Register sets are stored in statically which means we have to honor the order as @labath  pointed out. Ideally register sets array should be a dynamic list with all currently available registerset inserted in the list.


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

https://reviews.llvm.org/D91422