[NFC][OpaquePtr] Avoid using PointerType::getElementType() in VectorUtils.cpp

[nikic]

llvm/lib/Analysis/VectorUtils.cpp

967

Depending on how common the pattern is, it might make sense to add getLoadStoreType to complement getLoadStorePointerOperand (and getLoadStoreAlignment and getLoadStoreAddressSpace).

[dblaikie]

llvm/lib/Analysis/VectorUtils.cpp

967

Yeah, +1. There's something like 40 calls to this function, so probably worth having an opaque pointer equivalent.

[aeubanks]

create and use getLoadStoreType()

[nikic]

LGTM

llvm/lib/Analysis/VectorUtils.cpp

void InterleavedAccessInfo::collectConstStrideAccesses(
   // blocks in the loop in reverse postorder (i.e., in a topological order).
   // Such an ordering will ensure that any load/store that may be executed
   // before a second load/store will precede the second load/store in
   // AccessStrideInfo.
   LoopBlocksDFS DFS(TheLoop);
   DFS.perform(LI);
   for (BasicBlock *BB : make_range(DFS.beginRPO(), DFS.endRPO()))
     for (auto &I : *BB) {
-      auto *LI = dyn_cast<LoadInst>(&I);
-      auto *SI = dyn_cast<StoreInst>(&I);
-      if (!LI && !SI)
+      Type *ElementTy;
+      Value *Ptr;
+      if (auto *LI = dyn_cast<LoadInst>(&I)) {
+        ElementTy = LI->getType();
+        Ptr = LI->getPointerOperand();
+      } else if (auto *SI = dyn_cast<StoreInst>(&I)) {
+        ElementTy = SI->getValueOperand()->getType();
+        Ptr = SI->getPointerOperand();
+      } else {
         continue;
+      }
 
-      Value *Ptr = getLoadStorePointerOperand(&I);
       // We don't check wrapping here because we don't know yet if Ptr will be
       // part of a full group or a group with gaps. Checking wrapping for all
       // pointers (even those that end up in groups with no gaps) will be overly
       // conservative. For full groups, wrapping should be ok since if we would
       // wrap around the address space we would do a memory access at nullptr
       // even without the transformation. The wrapping checks are therefore
       // deferred until after we've formed the interleaved groups.
       int64_t Stride = getPtrStride(PSE, Ptr, TheLoop, Strides,
                                     /*Assume=*/true, /*ShouldCheckWrap=*/false);
 
       const SCEV *Scev = replaceSymbolicStrideSCEV(PSE, Strides, Ptr);
-      PointerType *PtrTy = cast<PointerType>(Ptr->getType());
-      uint64_t Size = DL.getTypeAllocSize(PtrTy->getElementType());
+      uint64_t Size = DL.getTypeAllocSize(ElementTy);
       AccessStrideInfo[&I] = StrideDescriptor(Stride, Scev, Size,
                                               getLoadStoreAlignment(&I));
     }
 }
 
 // Analyze interleaved accesses and collect them into interleaved load and
 // store groups.
 //