[PATCH] D60811: [PowerPC] Fix wrong ElemSIze when calling isConsecutiveLS()

Zhang Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 29 19:00:23 PDT 2019


ZhangKang added a comment.

In D60811#1472361 <https://reviews.llvm.org/D60811#1472361>, @efriedma wrote:

> I guess that's right, technically; "getScalarSizeInBits() / 8" will round down, and getStoreSize() will round up.  But still, please fix the code so it's clear that check is happening.


I have submitted a NFC patch D61076 <https://reviews.llvm.org/D61076> to follow your suggestion.


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