[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