[PATCH] D60811: [PowerPC] Fix wrong ElemSIze when calling isConsecutiveLS()
Jinsong Ji via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 18 15:19:26 PDT 2019
jsji added a comment.
In D60811#1472300 <https://reviews.llvm.org/D60811#1472300>, @efriedma wrote:
> For vectors where the element type is not byte-sized, loads can't be "consecutive" in the sense of this function; the store size of `<2 x i1>` is one byte.
>
I think isConcecitiveLS will early return false too with this fix. getScalarSizeInBits() / 8 != Bytes.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60811/new/
https://reviews.llvm.org/D60811
More information about the llvm-commits
mailing list