From: Chris Lattner <sabre@nondot.org>
Date: Fri, 14 Oct 2011 22:50:21 +0000 (+0000)
Subject: constify array itself, don't iterate multiple times.
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=aeee2d3a297a2ca41a07d33285a75cee23ad97d9;p=oota-llvm.git

constify array itself, don't iterate multiple times.


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

diff --git a/lib/VMCore/Function.cpp b/lib/VMCore/Function.cpp
index ef0827b110e..1215e6a57ce 100644
--- a/lib/VMCore/Function.cpp
+++ b/lib/VMCore/Function.cpp
@@ -417,7 +417,7 @@ bool Function::hasAddressTaken(const User* *PutOffender) const {
 /// FIXME: Remove after <rdar://problem/8031714> is fixed.
 /// FIXME: Is the above FIXME valid?
 bool Function::callsFunctionThatReturnsTwice() const {
-  static const char *ReturnsTwiceFns[] = {
+  static const char *const ReturnsTwiceFns[] = {
     "_setjmp",
     "setjmp",
     "sigsetjmp",
@@ -442,7 +442,7 @@ bool Function::callsFunctionThatReturnsTwice() const {
     if (!F)
       continue;
     StringRef Name = F->getName();
-    for (unsigned J = 0; J < array_lengthof(ReturnsTwiceFns); ++J) {
+    for (unsigned J = 0, e = array_lengthof(ReturnsTwiceFns); J != e; ++J) {
       if (Name == ReturnsTwiceFns[J])
         return true;
     }