From: Quentin Colombet <qcolombet@apple.com>
Date: Fri, 13 Oct 2017 21:16:13 +0000 (+0000)
Subject: [LegalizerInfo] Don't evaluate end boundary every time through the loop
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2cfe54276cf61d7cb275603dd6595c585897beb8;p=llvm

[LegalizerInfo] Don't evaluate end boundary every time through the loop

Match the LLVM coding standard for loop conditions.

NFC.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@315757 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/CodeGen/GlobalISel/LegalizerInfo.cpp b/lib/CodeGen/GlobalISel/LegalizerInfo.cpp
index 4fc4fa7d3eb..e7a46eadb44 100644
--- a/lib/CodeGen/GlobalISel/LegalizerInfo.cpp
+++ b/lib/CodeGen/GlobalISel/LegalizerInfo.cpp
@@ -58,7 +58,7 @@ LegalizerInfo::LegalizerInfo() {
 
 void LegalizerInfo::computeTables() {
   for (unsigned Opcode = 0; Opcode <= LastOp - FirstOp; ++Opcode) {
-    for (unsigned Idx = 0; Idx != Actions[Opcode].size(); ++Idx) {
+    for (unsigned Idx = 0, End = Actions[Opcode].size(); Idx != End; ++Idx) {
       for (auto &Action : Actions[Opcode][Idx]) {
         LLT Ty = Action.first;
         if (!Ty.isVector())
@@ -145,8 +145,9 @@ std::tuple<LegalizerInfo::LegalizeAction, unsigned, LLT>
 LegalizerInfo::getAction(const MachineInstr &MI,
                          const MachineRegisterInfo &MRI) const {
   SmallBitVector SeenTypes(8);
-  const MCOperandInfo *OpInfo = MI.getDesc().OpInfo;
-  for (unsigned i = 0; i < MI.getDesc().getNumOperands(); ++i) {
+  const MCInstrDesc &MCID = MI.getDesc();
+  const MCOperandInfo *OpInfo = MCID.OpInfo;
+  for (unsigned i = 0, e = MCID.getNumOperands(); i != e; ++i) {
     if (!OpInfo[i].isGenericType())
       continue;