[PATCH] D26014: [ConstantFold] Get the correct vector type when folding a getelementptr instruction with vector indices

[David Majnemer via llvm-commits]

majnemer added inline comments.


================
Comment at: lib/IR/ConstantFold.cpp:2063-2069
     if (VectorType *VT = dyn_cast<VectorType>(C->getType()))
       GEPTy = VectorType::get(GEPTy, VT->getNumElements());
+    // When the getelementptr has one or more vector indices, it will
+    // return a vector of pointers.
+    else
+      GEPTy = GetElementPtrInst::getGEPReturnType(C,
+        makeArrayRef((Value * const *)Idxs.data(), Idxs.size()));
----------------
Do we still need this if/else now that we are using `GetElementPtrInst::getGEPReturnType`?


https://reviews.llvm.org/D26014