diff options
| author | Med Ismail Bennani <medismail.bennani@gmail.com> | 2022-01-18 12:52:24 +0100 |
|---|---|---|
| committer | Med Ismail Bennani <medismail.bennani@gmail.com> | 2022-01-24 20:25:54 +0100 |
| commit | 45148bfe8aece6ca319dcc32351e20bba26b2ea7 (patch) | |
| tree | 1d4a982e3b7be8f9df51a0fd649108bbef8b02db /lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py | |
| parent | cfa55bfe781474a30467b1bbf2e7874985171196 (diff) | |
[lldb/Plugins] Fix ScriptedThread IndexID reporting
When listing all the Scripted Threads of a ScriptedProcess, we can see that all
have the thread index set to 1. This is caused by the lldb_private::Thread
constructor, which sets the m_index_id member using the provided thread id `tid`.
Because the call to the super constructor is done before instantiating
the `ScriptedThreadInterface`, lldb can't fetch the thread id from the
script instance, so it uses `LLDB_INVALID_THREAD_ID` instead.
To mitigate this, this patch takes advantage of the `ScriptedThread::Create`
fallible constructor idiom to defer calling the `ScriptedThread` constructor
(and the `Thread` super constructor with it), until we can fetch a valid
thread id `tid` from the `ScriptedThreadInterface`.
rdar://87432065
Differential Revision: https://reviews.llvm.org/D117076
Signed-off-by: Med Ismail Bennani <medismail.bennani@gmail.com>
Diffstat (limited to 'lldb/test/API/functionalities/scripted_process/TestScriptedProcess.py')
0 files changed, 0 insertions, 0 deletions