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

[Jeroen Dobbelaere via Phabricator via llvm-commits]

jeroen.dobbelaere added a comment.

Not sure how you can construct a test that triggers this problem, but in our out-of-tree version, this triggered problems.



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



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114141