[llvm] TargetLibraryInfo: Use pointer index size to determine getSizeTSize(). (PR #118747)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 5 05:57:33 PST 2024
================
@@ -1465,13 +1465,19 @@ unsigned TargetLibraryInfoImpl::getSizeTSize(const Module &M) const {
// Historically LLVM assume that size_t has same size as intptr_t (hence
// deriving the size from sizeof(int*) in address space zero). This should
- // work for most targets. For future consideration: DataLayout also implement
- // getIndexSizeInBits which might map better to size_t compared to
- // getPointerSizeInBits. Hard coding address space zero here might be
- // unfortunate as well. Maybe getDefaultGlobalsAddressSpace() or
- // getAllocaAddrSpace() is better.
+ // work for most targets. For future consideration: Hard coding address space
+ // zero here might be unfortunate. Maybe getMaxIndexSizeInBits() is better.
unsigned AddressSpace = 0;
- return M.getDataLayout().getPointerSizeInBits(AddressSpace);
+ return M.getDataLayout().getIndexSizeInBits(AddressSpace);
----------------
arsenm wrote:
If you're going to hardcode it, better to avoid the variable. This looks like it isn't hardcoded when grepping around for hardcoded cases
https://github.com/llvm/llvm-project/pull/118747
More information about the llvm-commits
mailing list