[llvm] [CodeGen] Remove extraneous casts to VariableID (NFC) (PR #145144)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 20 20:29:51 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/145144
We do not need these casts because values being cast here are already
of type VariableID.
>From 8c75d44bc6689b657f11263baad2364429f7e8be Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Thu, 19 Jun 2025 16:31:32 -0700
Subject: [PATCH] [CodeGen] Remove extraneous casts to VariableID (NFC)
We do not need these casts because values being cast here are already
of type VariableID.
---
llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp b/llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
index ffdf08eec9963..7da01e331b5a6 100644
--- a/llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
+++ b/llvm/lib/CodeGen/AssignmentTrackingAnalysis.cpp
@@ -1312,7 +1312,7 @@ class AssignmentTrackingLowering {
DenseSet<DebugAggregate> NotAlwaysStackHomed;
VariableID getVariableID(const DebugVariable &Var) {
- return static_cast<VariableID>(FnVarLocs->insertVariable(Var));
+ return FnVarLocs->insertVariable(Var);
}
/// Join the LiveOut values of preds that are contained in \p Visited into
@@ -1556,7 +1556,7 @@ void AssignmentTrackingLowering::emitDbgValue(
VariableID Var = getVariableID(DebugVariable(Source));
VarLocInfo VarLoc;
- VarLoc.VariableID = static_cast<VariableID>(Var);
+ VarLoc.VariableID = Var;
VarLoc.Expr = Expr;
VarLoc.Values = RawLocationWrapper(Val);
VarLoc.DL = DL;
@@ -1642,7 +1642,7 @@ void AssignmentTrackingLowering::processUnknownStoreToVariable(
Fn.getContext(), 0, 0, V.getVariable()->getScope(), InlinedAt);
VarLocInfo VarLoc;
- VarLoc.VariableID = static_cast<VariableID>(Var);
+ VarLoc.VariableID = Var;
VarLoc.Expr = DIExpression::get(I.getContext(), {});
VarLoc.Values = RawLocationWrapper(
ValueAsMetadata::get(PoisonValue::get(Type::getInt1Ty(I.getContext()))));
More information about the llvm-commits
mailing list