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

Joseph Huber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 3 13:33:45 PDT 2022


jhuber6 added a comment.

In D129813#3694683 <https://reviews.llvm.org/D129813#3694683>, @rampitec wrote:

> I am not sure about D120586 <https://reviews.llvm.org/D120586> timeline and capabilities, but since this was practically done I have finished the patch to a working state.

Like I said, the plan for that one was to just use the Attributor's `getAssumedUnderlyingObjects` to interproceduraly identify pointers with a common address space. We would then let the existing pass handle intraprocedural propagation. I'm not sure if that's an explicit replacement for the functionality proposed here. I think @jdoerfert wants to have address space deduction available inside the Attributor regardless, so we'll probably get that working eventually anyway. I don't have a definite timeline on this, but I could try to implement it sooner if this is important.


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

https://reviews.llvm.org/D129813



More information about the llvm-commits mailing list