From: Kazu Hirata Date: Mon, 29 Aug 2022 00:35:04 +0000 (-0700) Subject: [llvm] Use range-based for loops (NFC) X-Git-Tag: upstream/17.0.6~35113 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c63f8238751b83a4d1b87e1f7076b0d9ce71d399;p=platform%2Fupstream%2Fllvm.git [llvm] Use range-based for loops (NFC) --- diff --git a/llvm/lib/ObjCopy/MachO/MachOWriter.cpp b/llvm/lib/ObjCopy/MachO/MachOWriter.cpp index bc63328..84db941 100644 --- a/llvm/lib/ObjCopy/MachO/MachOWriter.cpp +++ b/llvm/lib/ObjCopy/MachO/MachOWriter.cpp @@ -302,9 +302,8 @@ void MachOWriter::writeSymbolTable() { .MachOLoadCommand.symtab_command_data; char *SymTable = (char *)Buf->getBufferStart() + SymTabCommand.symoff; - for (auto Iter = O.SymTable.Symbols.begin(), End = O.SymTable.Symbols.end(); - Iter != End; Iter++) { - SymbolEntry *Sym = Iter->get(); + for (auto &Symbol : O.SymTable.Symbols) { + SymbolEntry *Sym = Symbol.get(); uint32_t Nstrx = LayoutBuilder.getStringTableBuilder().getOffset(Sym->Name); if (Is64Bit) diff --git a/llvm/lib/Target/ARM/A15SDOptimizer.cpp b/llvm/lib/Target/ARM/A15SDOptimizer.cpp index 65da95b0..9bced06 100644 --- a/llvm/lib/Target/ARM/A15SDOptimizer.cpp +++ b/llvm/lib/Target/ARM/A15SDOptimizer.cpp @@ -290,11 +290,11 @@ unsigned A15SDOptimizer::optimizeSDPattern(MachineInstr *MI) { unsigned NumImplicit = 0, NumTotal = 0; unsigned NonImplicitReg = ~0U; - for (unsigned I = 1; I < MI->getNumExplicitOperands(); ++I) { - if (!MI->getOperand(I).isReg()) + for (MachineOperand &MO : llvm::drop_begin(MI->explicit_operands())) { + if (!MO.isReg()) continue; ++NumTotal; - Register OpReg = MI->getOperand(I).getReg(); + Register OpReg = MO.getReg(); if (!Register::isVirtualRegister(OpReg)) break; @@ -305,7 +305,7 @@ unsigned A15SDOptimizer::optimizeSDPattern(MachineInstr *MI) { if (Def->isImplicitDef()) ++NumImplicit; else - NonImplicitReg = MI->getOperand(I).getReg(); + NonImplicitReg = MO.getReg(); } if (NumImplicit == NumTotal - 1) diff --git a/llvm/lib/Target/SystemZ/SystemZLDCleanup.cpp b/llvm/lib/Target/SystemZ/SystemZLDCleanup.cpp index 1e6f971..8073ed0 100644 --- a/llvm/lib/Target/SystemZ/SystemZLDCleanup.cpp +++ b/llvm/lib/Target/SystemZ/SystemZLDCleanup.cpp @@ -105,8 +105,8 @@ bool SystemZLDCleanup::VisitNode(MachineDomTreeNode *Node, } // 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 &N : *Node) + Changed |= VisitNode(N, TLSBaseAddrReg); return Changed; } diff --git a/llvm/lib/Target/X86/X86InterleavedAccess.cpp b/llvm/lib/Target/X86/X86InterleavedAccess.cpp index 40174a1..8c31cf0 100644 --- a/llvm/lib/Target/X86/X86InterleavedAccess.cpp +++ b/llvm/lib/Target/X86/X86InterleavedAccess.cpp @@ -264,10 +264,10 @@ static void genShuffleBland(MVT VT, ArrayRef Mask, assert(VT.getSizeInBits() >= 256 && "This function doesn't accept width smaller then 256"); unsigned NumOfElm = VT.getVectorNumElements(); - for (unsigned i = 0; i < Mask.size(); i++) - Out.push_back(Mask[i] + LowOffset); - for (unsigned i = 0; i < Mask.size(); i++) - Out.push_back(Mask[i] + HighOffset + NumOfElm); + for (int I : Mask) + Out.push_back(I + LowOffset); + for (int I : Mask) + Out.push_back(I + HighOffset + NumOfElm); } // reorderSubVector returns the data to is the original state. And de-facto is diff --git a/llvm/lib/Transforms/InstCombine/InstCombineVectorOps.cpp b/llvm/lib/Transforms/InstCombine/InstCombineVectorOps.cpp index bac1dd2..4467a4f 100644 --- a/llvm/lib/Transforms/InstCombine/InstCombineVectorOps.cpp +++ b/llvm/lib/Transforms/InstCombine/InstCombineVectorOps.cpp @@ -1699,8 +1699,8 @@ static bool canEvaluateShuffled(Value *V, ArrayRef Mask, // Verify that 'CI' does not occur twice in Mask. A single 'insertelement' // can't put an element into multiple indices. bool SeenOnce = false; - for (int i = 0, e = Mask.size(); i != e; ++i) { - if (Mask[i] == ElementNumber) { + for (int I : Mask) { + if (I == ElementNumber) { if (SeenOnce) return false; SeenOnce = true;