Don't call getForwardedType() twice, as recommended by Chris.
authorJohn Criswell <criswell@uiuc.edu>
Thu, 6 May 2004 22:15:47 +0000 (22:15 +0000)
committerJohn Criswell <criswell@uiuc.edu>
Thu, 6 May 2004 22:15:47 +0000 (22:15 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@13391 91177308-0d34-0410-b5e6-96231b3b80d8

lib/VMCore/iMemory.cpp

index 32fe71a08626533af70bbacfc9ef03465fbb62ab..da4cc7483f0dff2b7832e2e6c39937ce14b6d6d3 100644 (file)
@@ -161,8 +161,8 @@ const Type* GetElementPtrInst::getIndexedType(const Type *Ptr,
     // of being refined to another type (and hence, may have dropped all
     // references to what it was using before).  So, use the new forwarded
     // type.
-    if (Ptr->getForwardedType()) {
-      Ptr = Ptr->getForwardedType();
+    if (const Type * Ty = Ptr->getForwardedType()) {
+      Ptr = Ty;
     }
   }
   return CurIdx == Idx.size() ? Ptr : 0;