[PATCH] D51203: AMDGPU: Handle 32-bit address wraparounds for SMRD opcodes

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 27 06:55:10 PDT 2018


nhaehnle added a comment.

Looking at SelectionDAGBuilder::visitGetElementPtr, nuw is set under certain conditions for `inbounds` getelementptr. I suspect we should be able to make most GEPs inbounds in Mesa - it just means that we never, not even temporarily, try to take addresses outside of properly allocated memory objects (buffers, arrays of descriptors).

Would the combination of:

- check NUW here
- create inbounds GEP

make good use of SMEM/SMRD immediates?


Repository:
  rL LLVM

https://reviews.llvm.org/D51203





More information about the llvm-commits mailing list