[Lldb-commits] [PATCH] D118055: [lldb] [gdb-remote] Support getting siginfo via API

Michał Górny via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jan 27 13:39:07 PST 2022


mgorny added a comment.

In D118055#3276558 <https://reviews.llvm.org/D118055#3276558>, @jingham wrote:

> We don't return errors separately for any of the API's that return an SBValue.  SBValue's carry their error with them (SBValue.GetError()) so it would be confusing to have two errors.

I'm sorry but I can't figure out how to set the error inside `SBValue`. The only user-facing thing that seems to allow passing `Status` seems to be `ValueObjectConstResult::Create()` but it feels like it's not supposed to be used like this (and it doesn't seem to preserve my `Status` either).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118055



More information about the lldb-commits mailing list