Skip to content

Commit c0d4b44

Browse files
committed
[DAG] DAGCombiner::tryStoreMergeOfLoads - remove unused StartAddress variable. NFCI.
Noticed in "initialization is never read" clang-tidy warning - the only StartAddress set/used is inside the load combine loop.
1 parent 9553655 commit c0d4b44

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17281,8 +17281,6 @@ bool DAGCombiner::tryStoreMergeOfLoads(SmallVectorImpl<MemOpLink> &StoreNodes,
1728117281
unsigned NumMemElts = MemVT.isVector() ? MemVT.getVectorNumElements() : 1;
1728217282
bool MadeChange = false;
1728317283

17284-
int64_t StartAddress = StoreNodes[0].OffsetFromBase;
17285-
1728617284
// Look for load nodes which are used by the stored values.
1728717285
SmallVector<MemOpLink, 8> LoadNodes;
1728817286

@@ -17350,7 +17348,7 @@ bool DAGCombiner::tryStoreMergeOfLoads(SmallVectorImpl<MemOpLink> &StoreNodes,
1735017348
unsigned LastLegalIntegerType = 1;
1735117349
bool isDereferenceable = true;
1735217350
bool DoIntegerTruncate = false;
17353-
StartAddress = LoadNodes[0].OffsetFromBase;
17351+
int64_t StartAddress = LoadNodes[0].OffsetFromBase;
1735417352
SDValue LoadChain = FirstLoad->getChain();
1735517353
for (unsigned i = 1; i < LoadNodes.size(); ++i) {
1735617354
// All loads must share the same chain.

0 commit comments

Comments
 (0)