[Lldb-commits] [PATCH] D103375: [lldb/API] Expose triple for SBProcessInfo.

2021-06-01 Thread Bruce Mitchener via Phabricator via lldb-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG251a5d9d5239: [lldb/API] Expose triple for SBProcessInfo. (authored by brucem). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

[Lldb-commits] [PATCH] D103375: [lldb/API] Expose triple for SBProcessInfo.

2021-06-01 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103375/new/ https://reviews.llvm.org/D103375 ___

[Lldb-commits] [PATCH] D103375: [lldb/API] Expose triple for SBProcessInfo.

2021-05-30 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor added inline comments. Comment at: lldb/source/API/SBProcessInfo.cpp:191 + // the string + triple = ConstString(arch.GetTriple().getTriple().c_str()).GetCString(); +} brucem wrote: > teemperor wrote: > > You need to 'instrument' these func

[Lldb-commits] [PATCH] D103375: [lldb/API] Expose triple for SBProcessInfo.

2021-05-30 Thread Bruce Mitchener via Phabricator via lldb-commits
brucem added inline comments. Comment at: lldb/source/API/SBProcessInfo.cpp:191 + // the string + triple = ConstString(arch.GetTriple().getTriple().c_str()).GetCString(); +} teemperor wrote: > You need to 'instrument' these functions. there is a too

[Lldb-commits] [PATCH] D103375: [lldb/API] Expose triple for SBProcessInfo.

2021-05-30 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor added inline comments. Comment at: lldb/source/API/SBProcessInfo.cpp:191 + // the string + triple = ConstString(arch.GetTriple().getTriple().c_str()).GetCString(); +} You need to 'instrument' these functions. there is a tool called `lldb-i

[Lldb-commits] [PATCH] D103375: [lldb/API] Expose triple for SBProcessInfo.

2021-05-30 Thread Bruce Mitchener via Phabricator via lldb-commits
brucem created this revision. brucem added a reviewer: lldb-commits. brucem requested review of this revision. Herald added a project: LLDB. This is present when doing a `platform process list` and is tracked by the underlying code. To do something like the process list via the SB API in the futur