[llvm] [RemoveDIs] Fix remapping of DbgLabelRecords. (PR #91447)
Harald van Dijk via llvm-commits
llvm-commits at lists.llvm.org
Wed May 8 01:42:25 PDT 2024
https://github.com/hvdijk created https://github.com/llvm/llvm-project/pull/91447
We already remapped DILocations for DbgVariableRecords, but DbgLabelRecords have debug locations too that need to be mapped the same way.
>From adc97ef5d8b3097cecc611f0e8727dcae6c7dffd Mon Sep 17 00:00:00 2001
From: Harald van Dijk <harald.vandijk at codeplay.com>
Date: Wed, 8 May 2024 09:36:33 +0100
Subject: [PATCH] [RemoveDIs] Fix remapping of DbgLabelRecords.
We already remapped DILocations for DbgVariableRecords, but
DbgLabelRecords have debug locations too that need to be mapped the same
way.
---
llvm/lib/Transforms/Utils/ValueMapper.cpp | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/llvm/lib/Transforms/Utils/ValueMapper.cpp b/llvm/lib/Transforms/Utils/ValueMapper.cpp
index 6ebdd85d37b43..1c877ee937fbf 100644
--- a/llvm/lib/Transforms/Utils/ValueMapper.cpp
+++ b/llvm/lib/Transforms/Utils/ValueMapper.cpp
@@ -538,17 +538,20 @@ Value *Mapper::mapValue(const Value *V) {
}
void Mapper::remapDbgRecord(DbgRecord &DR) {
+ // Remap DILocations.
+ auto *MappedDILoc = mapMetadata(DR.getDebugLoc());
+ DR.setDebugLoc(DebugLoc(cast<DILocation>(MappedDILoc)));
+
if (DbgLabelRecord *DLR = dyn_cast<DbgLabelRecord>(&DR)) {
+ // Remap labels.
DLR->setLabel(cast<DILabel>(mapMetadata(DLR->getLabel())));
return;
}
DbgVariableRecord &V = cast<DbgVariableRecord>(DR);
- // Remap variables and DILocations.
+ // Remap variables.
auto *MappedVar = mapMetadata(V.getVariable());
- auto *MappedDILoc = mapMetadata(V.getDebugLoc());
V.setVariable(cast<DILocalVariable>(MappedVar));
- V.setDebugLoc(DebugLoc(cast<DILocation>(MappedDILoc)));
bool IgnoreMissingLocals = Flags & RF_IgnoreMissingLocals;
More information about the llvm-commits
mailing list