[PATCH] D129813: [AMDGPU] Add IPO pass to infer pointer argument address spaces.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 16:40:54 PST 2022


arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.

This comes up often enough that I'd like to see it pushed to a useful place



================
Comment at: llvm/lib/Transforms/IPO/InferArgumentAddressSpaces.cpp:72
+// Return address space of a pointer \p V if we can deduce it or -1.
+static unsigned getPointerAddressSpace(const Value *V) {
+  for ( ; ; ) {
----------------
This is going to end up repeating all of InferAddressSpacesImpl::collectFlatAddressExpressions


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

https://reviews.llvm.org/D129813



More information about the llvm-commits mailing list