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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 18 15:34:20 PDT 2019


efriedma added a comment.

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.


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