[PATCH] D53162: [DataLayout] Add bit width of pointers to global values

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle created this revision.
nhaehnle added reviewers: arsenm, theraven, craig.topper, sanjoy, delena.
Herald added subscribers: tpr, wdng, jvesely.
nhaehnle added dependencies: D53161: Fix some cases where the index size was used instead of the pointer size, D53160: AMDGPU: Restrict DS load/store vectorizing on SI.

Together with leading known-zero bit tracking for getelementptr, this
recovers most of the code quality that was lost for AMDGPU's SI subtarget
with a previous commit.

AMDGPU has LDS (local) and GDS (region) address spaces that use 32 bit
pointers, even though the underlying memory is only 64 kiB, and therefore
pointers to global values always fit into 16 bits.

This recovers a lot of the code quality lost in "AMDGPU: Restrict DS
load/store vectorizing on SI".

Change-Id: Ic17f297b714039ea4988581b3abfafc57d11a6bd


Repository:
  rL LLVM

https://reviews.llvm.org/D53162

Files:
  docs/LangRef.rst
  include/llvm/IR/DataLayout.h
  lib/Analysis/ValueTracking.cpp
  lib/IR/DataLayout.cpp
  lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
  test/Transforms/LoadStoreVectorizer/AMDGPU/ds-bounds.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53162.169272.patch
Type: text/x-patch
Size: 11441 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181011/9ffce28b/attachment.bin>