[PATCH] D122758: DataLayout::getPointerSize() should always return a power of 2

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 30 14:04:09 PDT 2022


arsenm added a comment.

I don't think rounding this to a power of 2 fundamentally makes sense, and uses relying on that behavior should be adjusted to do something else.

I would also generally be suspicious of places using getPointerSize that really care about memory as it doesn't account for the declared alignment


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122758/new/

https://reviews.llvm.org/D122758



More information about the llvm-commits mailing list