[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