[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