[PATCH] D114141: Allow DataLayout to support arbitrary pointer sizes

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 14 00:30:59 PST 2021


uabelho added inline comments.


================
Comment at: llvm/lib/IR/DataLayout.cpp:715
+    MaxIndexSize =
+        std::max(MaxIndexSize, (unsigned)divideCeil(P.TypeBitWidth, 8));
 
----------------
jeroen.dobbelaere wrote:
> Shouldn't this be `.....divideCeil(P.IndexBitWidth, 8)` ?
> 
Ping @stephenneuendorffer 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114141



More information about the llvm-commits mailing list