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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 16:13:03 PDT 2022


arsenm added a comment.

@jdoerfert was telling me someone is already working on this



================
Comment at: llvm/lib/Transforms/IPO/InferArgumentAddressSpaces.cpp:139
+                      << "' to address space " << I.second << '\n');
+
+    PointerType *PT = dyn_cast<PointerType>(Arg->getType());
----------------
It would be better to replace the actual argument list with the new type too, and update all the users


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

https://reviews.llvm.org/D129813



More information about the llvm-commits mailing list