[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