This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG8a203bb22d16: [trace] rename ThreadIntelPT into TraceTrace
(authored by Walter Erquinigo ).
Repository:
rG LLVM Github Monorepo
C
vsk added inline comments.
Comment at: lldb/include/lldb/Target/ThreadTrace.h:34
+ /// this thread was being executed.
+ ThreadTrace(Process &process, lldb::tid_t tid, const FileSpec trace_file)
+ : Thread(process, tid), m_trace_file(trace_file) {}
wal
wallace added inline comments.
Comment at: lldb/include/lldb/Target/ThreadTrace.h:34
+ /// this thread was being executed.
+ ThreadTrace(Process &process, lldb::tid_t tid, const FileSpec trace_file)
+ : Thread(process, tid), m_trace_file(trace_file) {}
wallace updated this revision to Diff 298766.
wallace added a comment.
address comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89408/new/
https://reviews.llvm.org/D89408
Files:
lldb/include/lldb/Target/ProcessTrace.h
lldb/include/lldb/Ta
vsk added inline comments.
Comment at: lldb/include/lldb/Target/ThreadTrace.h:34
+ /// this thread was being executed.
+ ThreadTrace(Process &process, lldb::tid_t tid, const FileSpec trace_file)
+ : Thread(process, tid), m_trace_file(trace_file) {}
`co
clayborg added a comment.
LGTM. Pavel?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89408/new/
https://reviews.llvm.org/D89408
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llv