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

Marek Olšák via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 27 11:32:25 PDT 2018


mareko added a comment.

We can ignore old Mesa + new LLVM, because LLVM 7 is the first release to have 32-bit pointers, and I think we can fix that before release.

For internal driver additions, e.g. offset + small constant, the driver can use NUW or NSW. For app additions, we would just use normal Add. As long as NUW or NSW is preserved, we should be fine.


Repository:
  rL LLVM

https://reviews.llvm.org/D51203





More information about the llvm-commits mailing list