[PATCH] D40339: Use getStoreSize() in various places instead of BitSize >> 3

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 22 03:51:36 PST 2017


uabelho added a comment.

As a side note: This also helps people like myself that live with an out-of-tree target with
non-8bit bytes. We've changed getStoreSize() so it takes the byte size into account, so
using that instead of all these "/ 8" and ">> 3" all over the place makes life easier for us as
well. Thanks for that Jonas!


https://reviews.llvm.org/D40339





More information about the llvm-commits mailing list