[clang] [CGOpenMPRuntime] Use DenseMap::operator[] (NFC) (PR #107185)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 3 22:58:45 PDT 2024
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/107185
I'm planning to deprecate DenseMap::FindAndConstruct in favor of
DenseMap::operator[].
>From 24088d0b17e610039191537abfb196732a5e3218 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Tue, 3 Sep 2024 21:42:55 -0700
Subject: [PATCH] [CGOpenMPRuntime] Use DenseMap::operator[] (NFC)
I'm planning to deprecate DenseMap::FindAndConstruct in favor of
DenseMap::operator[].
---
clang/lib/CodeGen/CGOpenMPRuntime.cpp | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/clang/lib/CodeGen/CGOpenMPRuntime.cpp b/clang/lib/CodeGen/CGOpenMPRuntime.cpp
index 23b977be816025..2c5e2a0a1695d8 100644
--- a/clang/lib/CodeGen/CGOpenMPRuntime.cpp
+++ b/clang/lib/CodeGen/CGOpenMPRuntime.cpp
@@ -1327,25 +1327,25 @@ llvm::Function *CGOpenMPRuntime::emitTaskOutlinedFunction(
void CGOpenMPRuntime::setLocThreadIdInsertPt(CodeGenFunction &CGF,
bool AtCurrentPoint) {
- auto &Elem = OpenMPLocThreadIDMap.FindAndConstruct(CGF.CurFn);
- assert(!Elem.second.ServiceInsertPt && "Insert point is set already.");
+ auto &Elem = OpenMPLocThreadIDMap[CGF.CurFn];
+ assert(!Elem.ServiceInsertPt && "Insert point is set already.");
llvm::Value *Undef = llvm::UndefValue::get(CGF.Int32Ty);
if (AtCurrentPoint) {
- Elem.second.ServiceInsertPt = new llvm::BitCastInst(
+ Elem.ServiceInsertPt = new llvm::BitCastInst(
Undef, CGF.Int32Ty, "svcpt", CGF.Builder.GetInsertBlock());
} else {
- Elem.second.ServiceInsertPt =
+ Elem.ServiceInsertPt =
new llvm::BitCastInst(Undef, CGF.Int32Ty, "svcpt");
- Elem.second.ServiceInsertPt->insertAfter(CGF.AllocaInsertPt);
+ Elem.ServiceInsertPt->insertAfter(CGF.AllocaInsertPt);
}
}
void CGOpenMPRuntime::clearLocThreadIdInsertPt(CodeGenFunction &CGF) {
- auto &Elem = OpenMPLocThreadIDMap.FindAndConstruct(CGF.CurFn);
- if (Elem.second.ServiceInsertPt) {
- llvm::Instruction *Ptr = Elem.second.ServiceInsertPt;
- Elem.second.ServiceInsertPt = nullptr;
+ auto &Elem = OpenMPLocThreadIDMap[CGF.CurFn];
+ if (Elem.ServiceInsertPt) {
+ llvm::Instruction *Ptr = Elem.ServiceInsertPt;
+ Elem.ServiceInsertPt = nullptr;
Ptr->eraseFromParent();
}
}
@@ -1441,18 +1441,18 @@ llvm::Value *CGOpenMPRuntime::getThreadID(CodeGenFunction &CGF,
// kmpc_global_thread_num(ident_t *loc).
// Generate thread id value and cache this value for use across the
// function.
- auto &Elem = OpenMPLocThreadIDMap.FindAndConstruct(CGF.CurFn);
- if (!Elem.second.ServiceInsertPt)
+ auto &Elem = OpenMPLocThreadIDMap[CGF.CurFn];
+ if (!Elem.ServiceInsertPt)
setLocThreadIdInsertPt(CGF);
CGBuilderTy::InsertPointGuard IPG(CGF.Builder);
- CGF.Builder.SetInsertPoint(Elem.second.ServiceInsertPt);
+ CGF.Builder.SetInsertPoint(Elem.ServiceInsertPt);
auto DL = ApplyDebugLocation::CreateDefaultArtificial(CGF, Loc);
llvm::CallInst *Call = CGF.Builder.CreateCall(
OMPBuilder.getOrCreateRuntimeFunction(CGM.getModule(),
OMPRTL___kmpc_global_thread_num),
emitUpdateLocation(CGF, Loc));
Call->setCallingConv(CGF.getRuntimeCC());
- Elem.second.ThreadID = Call;
+ Elem.ThreadID = Call;
return Call;
}
More information about the cfe-commits
mailing list