diff options
author | Nick Desaulniers <ndesaulniers@google.com> | 2019-05-20 22:17:43 +0000 |
---|---|---|
committer | Nick Desaulniers <ndesaulniers@google.com> | 2019-05-20 22:17:43 +0000 |
commit | 28e351af2ad29e0af4934e9286519b5e1863fecc (patch) | |
tree | e1a37fd2df8bf4cb3fbf707b839b7513229795b6 /llvm/lib/ExecutionEngine/Orc/Layer.cpp | |
parent | 6dd08e335ffc10520924d063323a8869acfb04d7 (diff) | |
download | bcm5719-llvm-28e351af2ad29e0af4934e9286519b5e1863fecc.tar.gz bcm5719-llvm-28e351af2ad29e0af4934e9286519b5e1863fecc.zip |
[ORC] fix use-after-move. NFC
Summary:
scan-build flagged a potential use-after-move in debug builds. It's not
safe that a moved from value contains anything but garbage. Manually
DRY up these repeated expressions.
Reviewers: lhames
Reviewed By: lhames
Subscribers: hiraditya, llvm-commits, srhines
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D62112
llvm-svn: 361203
Diffstat (limited to 'llvm/lib/ExecutionEngine/Orc/Layer.cpp')
-rw-r--r-- | llvm/lib/ExecutionEngine/Orc/Layer.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/llvm/lib/ExecutionEngine/Orc/Layer.cpp b/llvm/lib/ExecutionEngine/Orc/Layer.cpp index 73470bf18c5..3ed2dabf454 100644 --- a/llvm/lib/ExecutionEngine/Orc/Layer.cpp +++ b/llvm/lib/ExecutionEngine/Orc/Layer.cpp @@ -87,17 +87,15 @@ void BasicIRLayerMaterializationUnit::materialize( #ifndef NDEBUG auto &ES = R.getTargetJITDylib().getExecutionSession(); + auto &N = R.getTargetJITDylib().getName(); #endif // NDEBUG auto Lock = TSM.getContextLock(); - LLVM_DEBUG(ES.runSessionLocked([&]() { - dbgs() << "Emitting, for " << R.getTargetJITDylib().getName() << ", " - << *this << "\n"; - });); + LLVM_DEBUG(ES.runSessionLocked( + [&]() { dbgs() << "Emitting, for " << N << ", " << *this << "\n"; });); L.emit(std::move(R), std::move(TSM)); LLVM_DEBUG(ES.runSessionLocked([&]() { - dbgs() << "Finished emitting, for " << R.getTargetJITDylib().getName() - << ", " << *this << "\n"; + dbgs() << "Finished emitting, for " << N << ", " << *this << "\n"; });); } |