diff options
| author | Jonas Devlieghere <jonas@devlieghere.com> | 2019-04-03 21:31:22 +0000 |
|---|---|---|
| committer | Jonas Devlieghere <jonas@devlieghere.com> | 2019-04-03 21:31:22 +0000 |
| commit | 306809f292c9dd26e1a901b139f65976cdd5a4b2 (patch) | |
| tree | b22cc9af74e85c1ab680b7ad5a8ac035fda72f5e /lldb/source/API/SBThreadPlan.cpp | |
| parent | 7c711ccf36e0a818f76d711715956ffe831f53b6 (diff) | |
| download | llvm-306809f292c9dd26e1a901b139f65976cdd5a4b2.zip llvm-306809f292c9dd26e1a901b139f65976cdd5a4b2.tar.gz llvm-306809f292c9dd26e1a901b139f65976cdd5a4b2.tar.bz2 | |
[Reproducers] Capture return values of functions returning by ptr/ref
For some reason I had convinced myself that functions returning by
pointer or reference do not require recording their result. However,
after further considering I don't see how that could work, at least not
with the current implementation. Interestingly enough, the reproducer
instrumentation already (mostly) accounts for this, though the
lldb-instr tool did not.
This patch adds the missing macros and updates the lldb-instr tool.
Differential revision: https://reviews.llvm.org/D60178
llvm-svn: 357639
Diffstat (limited to 'lldb/source/API/SBThreadPlan.cpp')
| -rw-r--r-- | lldb/source/API/SBThreadPlan.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lldb/source/API/SBThreadPlan.cpp b/lldb/source/API/SBThreadPlan.cpp index 05f813f..742a3570 100644 --- a/lldb/source/API/SBThreadPlan.cpp +++ b/lldb/source/API/SBThreadPlan.cpp @@ -82,7 +82,7 @@ const lldb::SBThreadPlan &SBThreadPlan::operator=(const SBThreadPlan &rhs) { if (this != &rhs) m_opaque_sp = rhs.m_opaque_sp; - return *this; + return LLDB_RECORD_RESULT(*this); } //---------------------------------------------------------------------- // Destructor @@ -92,7 +92,7 @@ SBThreadPlan::~SBThreadPlan() {} lldb_private::ThreadPlan *SBThreadPlan::get() { LLDB_RECORD_METHOD_NO_ARGS(lldb_private::ThreadPlan *, SBThreadPlan, get); - return m_opaque_sp.get(); + return LLDB_RECORD_RESULT(m_opaque_sp.get()); } bool SBThreadPlan::IsValid() const { |