[PATCH] D143945: [AMDGPU] Add legalization case for PTR_ADD on buffer pointers

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 13 16:52:29 PST 2023


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

I guess. we can go with this for now. I think this doesn't really work the same, since e.g. what happens on overflow of the 48-bit pointer part?

We do need to be able to handle these standalone, independent of a buffer instruction which may require emulation of any bounds checking it performs?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143945



More information about the llvm-commits mailing list