Analysis: Remove redundant assertion
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Tue, 20 Sep 2022 13:26:32 +0000 (09:26 -0400)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Tue, 20 Sep 2022 13:39:45 +0000 (09:39 -0400)
This assert guards the same assertion inside getTypeStoreSizeInBits

llvm/lib/Analysis/Loads.cpp

index a21649a..9eff2b1 100644 (file)
@@ -82,9 +82,7 @@ static bool isDereferenceableAndAlignedPointer(
       // As we recursed through GEPs to get here, we've incrementally checked
       // that each step advanced by a multiple of the alignment. If our base is
       // properly aligned, then the original offset accessed must also be.
-      Type *Ty = V->getType();
-      assert(Ty->isSized() && "must be sized");
-      APInt Offset(DL.getTypeStoreSizeInBits(Ty), 0);
+      APInt Offset(DL.getTypeStoreSizeInBits(V->getType()), 0);
       return isAligned(V, Offset, Alignment, DL);
     }
 
@@ -174,9 +172,7 @@ static bool isDereferenceableAndAlignedPointer(
         // checked that each step advanced by a multiple of the alignment. If
         // our base is properly aligned, then the original offset accessed
         // must also be.
-        Type *Ty = V->getType();
-        assert(Ty->isSized() && "must be sized");
-        APInt Offset(DL.getTypeStoreSizeInBits(Ty), 0);
+        APInt Offset(DL.getTypeStoreSizeInBits(V->getType()), 0);
         return isAligned(V, Offset, Alignment, DL);
       }
     }