diff options
| author | Lang Hames <lhames@gmail.com> | 2017-01-29 00:51:17 +0000 |
|---|---|---|
| committer | Lang Hames <lhames@gmail.com> | 2017-01-29 00:51:17 +0000 |
| commit | ce15049ddd6d6cefb077e7af7314874daec93a9b (patch) | |
| tree | cda8e08d3810101c418cb1723c0c38ff594b1f05 | |
| parent | 56c0e2882725a157f444444c9b354c53326bf156 (diff) | |
| download | bcm5719-llvm-ce15049ddd6d6cefb077e7af7314874daec93a9b.tar.gz bcm5719-llvm-ce15049ddd6d6cefb077e7af7314874daec93a9b.zip | |
[Orc][RPC] Remove a couple of redundant calls to abandonAllPendingResponses.
appendCallAsync, which all RPC call functions ultimately build on, will call
abandonAllPendingResponses on channel error. These extra calls are redundant.
llvm-svn: 293405
| -rw-r--r-- | llvm/include/llvm/ExecutionEngine/Orc/RPCUtils.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/llvm/include/llvm/ExecutionEngine/Orc/RPCUtils.h b/llvm/include/llvm/ExecutionEngine/Orc/RPCUtils.h index 272b480565b..8dd396672ee 100644 --- a/llvm/include/llvm/ExecutionEngine/Orc/RPCUtils.h +++ b/llvm/include/llvm/ExecutionEngine/Orc/RPCUtils.h @@ -1155,7 +1155,6 @@ public: return Error::success(); }, Args...)) { - this->abandonPendingResponses(); RTraits::consumeAbandoned(FutureResult.get()); return std::move(Err); } @@ -1188,12 +1187,6 @@ public: typename detail::ResultTraits<AltRetT>::ErrorReturnType callB(const ArgTs &... Args) { if (auto FutureResOrErr = callNB<Func>(Args...)) { - if (auto Err = this->C.send()) { - this->abandonPendingResponses(); - detail::ResultTraits<typename Func::ReturnType>::consumeAbandoned( - std::move(FutureResOrErr->get())); - return std::move(Err); - } return FutureResOrErr->get(); } else return FutureResOrErr.takeError(); |

