[PATCH] D27319: Fix buildbot failure in non-assert-enabled -Werror build due to r288410

David Kreitzer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 13:08:59 PST 2016


DavidKreitzer created this revision.
DavidKreitzer added a reviewer: Farhana.
DavidKreitzer added a subscriber: llvm-commits.

The fix is simply to forward substitute the definition of VecTy into its uses.


https://reviews.llvm.org/D27319

Files:
  lib/Target/X86/X86InterleavedAccess.cpp


Index: lib/Target/X86/X86InterleavedAccess.cpp
===================================================================
--- lib/Target/X86/X86InterleavedAccess.cpp
+++ lib/Target/X86/X86InterleavedAccess.cpp
@@ -115,14 +115,12 @@
 bool X86InterleavedAccessGroup::decompose(
     Instruction *VecInst, unsigned NumSubVectors, VectorType *SubVecTy,
     SmallVectorImpl<Instruction *> &DecomposedVectors) {
-  Type *VecTy = VecInst->getType();
-
-  assert(VecTy->isVectorTy() &&
-         DL.getTypeSizeInBits(VecTy) >=
+  assert(VecInst->getType()->isVectorTy() &&
+         DL.getTypeSizeInBits(VecInst->getType()) >=
              DL.getTypeSizeInBits(SubVecTy) * NumSubVectors &&
          "Invalid Inst-size!!!");
-  assert(VecTy->getVectorElementType() == SubVecTy->getVectorElementType() &&
-         "Element type mismatched!!!");
+  assert(VecInst->getType()->getVectorElementType() ==
+         SubVecTy->getVectorElementType() && "Element type mismatched!!!");
 
   if (!isa<LoadInst>(VecInst))
     return false;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27319.79974.patch
Type: text/x-patch
Size: 1018 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161201/127d08c6/attachment.bin>


More information about the llvm-commits mailing list