[llvm] 925b7d6 - [ORC] Replace some KV loop variables with structured bindings.
Lang Hames via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 8 17:50:59 PDT 2024
Author: Lang Hames
Date: 2024-04-08T18:50:52-06:00
New Revision: 925b7d6f62bf8f0b204cc1cb24a4a75c7bc5e0ae
URL: https://github.com/llvm/llvm-project/commit/925b7d6f62bf8f0b204cc1cb24a4a75c7bc5e0ae
DIFF: https://github.com/llvm/llvm-project/commit/925b7d6f62bf8f0b204cc1cb24a4a75c7bc5e0ae.diff
LOG: [ORC] Replace some KV loop variables with structured bindings.
Same idea as 006aaf32258 -- reduce boilerplate and improve readability. This
time updates will be piecemeal to make it easier to identify errors.
Coding my way home: 2.18555S, 93.78063W
Added:
Modified:
llvm/lib/ExecutionEngine/Orc/Core.cpp
Removed:
################################################################################
diff --git a/llvm/lib/ExecutionEngine/Orc/Core.cpp b/llvm/lib/ExecutionEngine/Orc/Core.cpp
index 8bb68b45951db1..a9d1998e093054 100644
--- a/llvm/lib/ExecutionEngine/Orc/Core.cpp
+++ b/llvm/lib/ExecutionEngine/Orc/Core.cpp
@@ -87,13 +87,13 @@ FailedToMaterialize::FailedToMaterialize(
// FIXME: Use a new dep-map type for FailedToMaterialize errors so that we
// don't have to manually retain/release.
- for (auto &KV : *this->Symbols)
- KV.first->Retain();
+ for (auto &[JD, Syms] : *this->Symbols)
+ JD->Retain();
}
FailedToMaterialize::~FailedToMaterialize() {
- for (auto &KV : *Symbols)
- KV.first->Release();
+ for (auto &[JD, Syms] : *Symbols)
+ JD->Release();
}
std::error_code FailedToMaterialize::convertToErrorCode() const {
More information about the llvm-commits
mailing list