[ARM] Don't look through pointer types in canTailPredicateLoop()

[nikic]

Summary

Inspecting the pointer element type here is incompatible with opaque pointers, and doesn't seem necessary to me. I think the intention here might have been to check the type of load/store pointer arguments, but I believe those should get checked through their return type or value operand anyway. I don't get any test failures if I simply drop this.

[dmgreen]

I agree. LGTM

llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp

for (BasicBlock *BB : L->blocks()) {
     for (Instruction &I : BB->instructionsWithoutDebug()) {
       if (isa<PHINode>(&I))
         continue;
       if (!canTailPredicateInstruction(I, ICmpCount)) {
         LLVM_DEBUG(dbgs() << "Instruction not allowed: "; I.dump());
         return false;
       }
 
       Type *T  = I.getType();
-      if (T->isPointerTy())
-        T = T->getPointerElementType();
-
       if (T->getScalarSizeInBits() > 32) {
         LLVM_DEBUG(dbgs() << "Unsupported Type: "; T->dump());
         return false;
       }
       if (isa<StoreInst>(I) || isa<LoadInst>(I)) {
         Value *Ptr = getLoadStorePointerOperand(&I);
         Type *AccessTy = getLoadStoreType(&I);
         int64_t NextStride = getPtrStride(PSE, AccessTy, Ptr, L);