[llvm] Use range-based for loops (NFC)
GitOrigin-RevId: ba0407ba86d0a87f01e1a676f9dc23bae3e9c169
diff --git a/include/llvm/Analysis/LoopInfoImpl.h b/include/llvm/Analysis/LoopInfoImpl.h
index a96a698..ffe5617 100644
--- a/include/llvm/Analysis/LoopInfoImpl.h
+++ b/include/llvm/Analysis/LoopInfoImpl.h
@@ -292,17 +292,12 @@
getExitBlocks(ExitBBs);
df_iterator_default_set<BlockT *> VisitSet;
VisitSet.insert(ExitBBs.begin(), ExitBBs.end());
- df_ext_iterator<BlockT *, df_iterator_default_set<BlockT *>>
- BI = df_ext_begin(getHeader(), VisitSet),
- BE = df_ext_end(getHeader(), VisitSet);
// Keep track of the BBs visited.
SmallPtrSet<BlockT *, 8> VisitedBBs;
// Check the individual blocks.
- for (; BI != BE; ++BI) {
- BlockT *BB = *BI;
-
+ for (BlockT *BB : depth_first_ext(getHeader(), VisitSet)) {
assert(std::any_of(GraphTraits<BlockT *>::child_begin(BB),
GraphTraits<BlockT *>::child_end(BB),
[&](BlockT *B) { return contains(B); }) &&
diff --git a/include/llvm/Analysis/TargetTransformInfoImpl.h b/include/llvm/Analysis/TargetTransformInfoImpl.h
index 55b3b53..1364e7a 100644
--- a/include/llvm/Analysis/TargetTransformInfoImpl.h
+++ b/include/llvm/Analysis/TargetTransformInfoImpl.h
@@ -50,8 +50,8 @@
TTI::TargetCostKind CostKind) const {
// In the basic model, we just assume that all-constant GEPs will be folded
// into their uses via addressing modes.
- for (unsigned Idx = 0, Size = Operands.size(); Idx != Size; ++Idx)
- if (!isa<Constant>(Operands[Idx]))
+ for (const Value *Operand : Operands)
+ if (!isa<Constant>(Operand))
return TTI::TCC_Basic;
return TTI::TCC_Free;
diff --git a/include/llvm/Bitstream/BitstreamReader.h b/include/llvm/Bitstream/BitstreamReader.h
index 10a0a4e..420a9e9 100644
--- a/include/llvm/Bitstream/BitstreamReader.h
+++ b/include/llvm/Bitstream/BitstreamReader.h
@@ -55,10 +55,9 @@
if (!BlockInfoRecords.empty() && BlockInfoRecords.back().BlockID == BlockID)
return &BlockInfoRecords.back();
- for (unsigned i = 0, e = static_cast<unsigned>(BlockInfoRecords.size());
- i != e; ++i)
- if (BlockInfoRecords[i].BlockID == BlockID)
- return &BlockInfoRecords[i];
+ for (const BlockInfo &BI : BlockInfoRecords)
+ if (BI.BlockID == BlockID)
+ return &BI;
return nullptr;
}
diff --git a/include/llvm/Bitstream/BitstreamWriter.h b/include/llvm/Bitstream/BitstreamWriter.h
index 143f9ba..1475c32 100644
--- a/include/llvm/Bitstream/BitstreamWriter.h
+++ b/include/llvm/Bitstream/BitstreamWriter.h
@@ -267,10 +267,9 @@
if (!BlockInfoRecords.empty() && BlockInfoRecords.back().BlockID == BlockID)
return &BlockInfoRecords.back();
- for (unsigned i = 0, e = static_cast<unsigned>(BlockInfoRecords.size());
- i != e; ++i)
- if (BlockInfoRecords[i].BlockID == BlockID)
- return &BlockInfoRecords[i];
+ for (BlockInfo &BI : BlockInfoRecords)
+ if (BI.BlockID == BlockID)
+ return &BI;
return nullptr;
}
diff --git a/include/llvm/IR/ModuleSummaryIndexYAML.h b/include/llvm/IR/ModuleSummaryIndexYAML.h
index f09ee0e..74e9279 100644
--- a/include/llvm/IR/ModuleSummaryIndexYAML.h
+++ b/include/llvm/IR/ModuleSummaryIndexYAML.h
@@ -270,8 +270,8 @@
V.insert({GlobalValue::getGUID(Key), {std::string(Key), TId}});
}
static void output(IO &io, TypeIdSummaryMapTy &V) {
- for (auto TidIter = V.begin(); TidIter != V.end(); TidIter++)
- io.mapRequired(TidIter->second.first.c_str(), TidIter->second.second);
+ for (auto &TidIter : V)
+ io.mapRequired(TidIter.second.first.c_str(), TidIter.second.second);
}
};
diff --git a/lib/ExecutionEngine/MCJIT/MCJIT.cpp b/lib/ExecutionEngine/MCJIT/MCJIT.cpp
index 4ac901d..e75c4eb 100644
--- a/lib/ExecutionEngine/MCJIT/MCJIT.cpp
+++ b/lib/ExecutionEngine/MCJIT/MCJIT.cpp
@@ -659,9 +659,8 @@
static_cast<uint64_t>(reinterpret_cast<uintptr_t>(Obj.getData().data()));
std::lock_guard<sys::Mutex> locked(lock);
MemMgr->notifyObjectLoaded(this, Obj);
- for (unsigned I = 0, S = EventListeners.size(); I < S; ++I) {
- EventListeners[I]->notifyObjectLoaded(Key, Obj, L);
- }
+ for (JITEventListener *EL : EventListeners)
+ EL->notifyObjectLoaded(Key, Obj, L);
}
void MCJIT::notifyFreeingObject(const object::ObjectFile &Obj) {
diff --git a/lib/Target/AMDGPU/SIInstrInfo.cpp b/lib/Target/AMDGPU/SIInstrInfo.cpp
index 02384e3..0f4c778 100644
--- a/lib/Target/AMDGPU/SIInstrInfo.cpp
+++ b/lib/Target/AMDGPU/SIInstrInfo.cpp
@@ -2209,8 +2209,8 @@
}
}
- for (unsigned I = 3; I < MI.getNumExplicitOperands(); ++I)
- MovDPP.addImm(MI.getOperand(I).getImm());
+ for (const MachineOperand &MO : llvm::drop_begin(MI.explicit_operands(), 3))
+ MovDPP.addImm(MO.getImm());
Split[Part] = MovDPP;
++Part;
diff --git a/lib/Target/X86/X86CmovConversion.cpp b/lib/Target/X86/X86CmovConversion.cpp
index 8016960..765ff5a 100644
--- a/lib/Target/X86/X86CmovConversion.cpp
+++ b/lib/Target/X86/X86CmovConversion.cpp
@@ -437,8 +437,7 @@
// Depth-Diff[i]:
// Number of cycles saved in first 'i` iterations by optimizing the loop.
//===--------------------------------------------------------------------===//
- for (unsigned I = 0; I < LoopIterations; ++I) {
- DepthInfo &MaxDepth = LoopDepth[I];
+ for (DepthInfo &MaxDepth : LoopDepth) {
for (auto *MBB : Blocks) {
// Clear physical registers Def map.
RegDefMaps[PhyRegType].clear();
diff --git a/lib/Target/X86/X86FrameLowering.cpp b/lib/Target/X86/X86FrameLowering.cpp
index d524090..a4e27a6 100644
--- a/lib/Target/X86/X86FrameLowering.cpp
+++ b/lib/Target/X86/X86FrameLowering.cpp
@@ -1097,9 +1097,9 @@
for (MachineInstr &MI : *LoopMBB) {
MI.setFlag(MachineInstr::FrameSetup);
}
- for (MachineBasicBlock::iterator CMBBI = ContinueMBB->begin();
- CMBBI != ContinueMBBI; ++CMBBI) {
- CMBBI->setFlag(MachineInstr::FrameSetup);
+ for (MachineInstr &MI :
+ llvm::make_range(ContinueMBB->begin(), ContinueMBBI)) {
+ MI.setFlag(MachineInstr::FrameSetup);
}
}
}
diff --git a/lib/Target/X86/X86InstrInfo.cpp b/lib/Target/X86/X86InstrInfo.cpp
index 4904e79..81db055 100644
--- a/lib/Target/X86/X86InstrInfo.cpp
+++ b/lib/Target/X86/X86InstrInfo.cpp
@@ -9425,8 +9425,8 @@
}
// Visit the children of this block in the dominator tree.
- for (auto I = Node->begin(), E = Node->end(); I != E; ++I) {
- Changed |= VisitNode(*I, TLSBaseAddrReg);
+ for (auto &I : *Node) {
+ Changed |= VisitNode(I, TLSBaseAddrReg);
}
return Changed;
diff --git a/lib/Target/X86/X86VZeroUpper.cpp b/lib/Target/X86/X86VZeroUpper.cpp
index 59b5dc1..17fa942 100644
--- a/lib/Target/X86/X86VZeroUpper.cpp
+++ b/lib/Target/X86/X86VZeroUpper.cpp
@@ -299,9 +299,8 @@
bool YmmOrZmmUsed = FnHasLiveInYmmOrZmm;
for (auto *RC : {&X86::VR256RegClass, &X86::VR512_0_15RegClass}) {
if (!YmmOrZmmUsed) {
- for (TargetRegisterClass::iterator i = RC->begin(), e = RC->end(); i != e;
- i++) {
- if (!MRI.reg_nodbg_empty(*i)) {
+ for (MCPhysReg R : *RC) {
+ if (!MRI.reg_nodbg_empty(R)) {
YmmOrZmmUsed = true;
break;
}
diff --git a/lib/Transforms/IPO/GlobalOpt.cpp b/lib/Transforms/IPO/GlobalOpt.cpp
index 6df0409..2a48b44 100644
--- a/lib/Transforms/IPO/GlobalOpt.cpp
+++ b/lib/Transforms/IPO/GlobalOpt.cpp
@@ -140,9 +140,7 @@
case Type::StructTyID: {
StructType *STy = cast<StructType>(Ty);
if (STy->isOpaque()) return true;
- for (StructType::element_iterator I = STy->element_begin(),
- E = STy->element_end(); I != E; ++I) {
- Type *InnerTy = *I;
+ for (Type *InnerTy : STy->elements()) {
if (isa<PointerType>(InnerTy)) return true;
if (isa<StructType>(InnerTy) || isa<ArrayType>(InnerTy) ||
isa<VectorType>(InnerTy))