From 73cb576c35bd1368e00a2160f2cd935d70908e23 Mon Sep 17 00:00:00 2001
From: Aaron Ballman <aaron@aaronballman.com>
Date: Mon, 11 May 2015 13:10:17 +0000
Subject: [PATCH] Replacing a range-based for loop with an old-style for loop.
 This code was previously causing a warning with MSVC about a
 compiler-generated local variable because TargetRegistry::begin() and end()
 are static member functions. NFC.

llvm-svn: 236990
---
 llvm/unittests/Support/TargetRegistry.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/llvm/unittests/Support/TargetRegistry.cpp b/llvm/unittests/Support/TargetRegistry.cpp
index dd5ef431ae07..7738e5240940 100644
--- a/llvm/unittests/Support/TargetRegistry.cpp
+++ b/llvm/unittests/Support/TargetRegistry.cpp
@@ -23,8 +23,9 @@ TEST(TargetRegistry, TargetHasArchType) {
   llvm::InitializeAllTargetInfos();
 
   llvm::TargetRegistry RegistryRoot;
-  for (const auto &Target : RegistryRoot) {
-    StringRef Name = Target.getName();
+  for (auto &I = TargetRegistry::begin(), &E = TargetRegistry::end();
+       I != E; ++I) {
+    StringRef Name = I->getName();
     // There is really no way (at present) to ask a Target whether it targets
     // a specific architecture, because the logic for that is buried in a
     // predicate.