[llvm] [NFC] Rename variables to FPOp (PR #173792)
via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 28 10:54:37 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-selectiondag
Author: Mikołaj Piróg (mikolaj-pirog)
<details>
<summary>Changes</summary>
In fpext function the variables was wrongly named TrunInst -- this is obviously wrong. I've changed the name in both functions to generic FPOp
---
Full diff: https://github.com/llvm/llvm-project/pull/173792.diff
1 Files Affected:
- (modified) llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp (+4-4)
``````````diff
diff --git a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
index e35c0d95f3941..e948678f81d4c 100644
--- a/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
@@ -3996,8 +3996,8 @@ void SelectionDAGBuilder::visitFPTrunc(const User &I) {
SDValue N = getValue(I.getOperand(0));
SDLoc dl = getCurSDLoc();
SDNodeFlags Flags;
- if (auto *TruncInst = dyn_cast<FPMathOperator>(&I))
- Flags.copyFMF(*TruncInst);
+ if (auto *FPOp = dyn_cast<FPMathOperator>(&I))
+ Flags.copyFMF(*FPOp);
const TargetLowering &TLI = DAG.getTargetLoweringInfo();
EVT DestVT = TLI.getValueType(DAG.getDataLayout(), I.getType());
setValue(&I, DAG.getNode(ISD::FP_ROUND, dl, DestVT, N,
@@ -4012,8 +4012,8 @@ void SelectionDAGBuilder::visitFPExt(const User &I) {
EVT DestVT = DAG.getTargetLoweringInfo().getValueType(DAG.getDataLayout(),
I.getType());
SDNodeFlags Flags;
- if (auto *TruncInst = dyn_cast<FPMathOperator>(&I))
- Flags.copyFMF(*TruncInst);
+ if (auto *FPOp = dyn_cast<FPMathOperator>(&I))
+ Flags.copyFMF(*FPOp);
setValue(&I, DAG.getNode(ISD::FP_EXTEND, getCurSDLoc(), DestVT, N, Flags));
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/173792
More information about the llvm-commits
mailing list