[PATCH] D112370: [BasicAA] Use index size instead of pointer size

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 25 10:33:32 PDT 2021


reames added inline comments.


================
Comment at: llvm/lib/Analysis/BasicAliasAnalysis.cpp:557
 
-  unsigned MaxPointerSize = DL.getMaxPointerSizeInBits();
   DecomposedGEP Decomposed;
----------------
The use of maximum pointer/index size seems suspect here.  Shouldn't we be working in the bitwidth of a single AS at all times?  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112370



More information about the llvm-commits mailing list