[PATCH] D73144: [SVE] Pass Scalable argument to VectorType::get in Bitcode Reader

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 21 15:00:01 PST 2020


ctetreau marked an inline comment as done.
ctetreau added inline comments.


================
Comment at: llvm/lib/Bitcode/Reader/BitcodeReader.cpp:4175
       FullTy = VectorType::get(FullTy->getVectorElementType(),
-                               Mask->getType()->getVectorNumElements());
+                               Mask->getType()->getVectorElementCount());
       InstructionList.push_back(I);
----------------
This is the source of the bug from last patch. I passed the length of Vec1, not of the mask, which is not in line with the documented behavior of shufflevector or the original code.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73144/new/

https://reviews.llvm.org/D73144





More information about the llvm-commits mailing list