[PATCH] D134308: AtomicExpand: Use llvm.ptrmask instead of ptrtoint

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D134308#3808877 <https://reviews.llvm.org/D134308#3808877>, @arsenm wrote:

> In D134308#3808841 <https://reviews.llvm.org/D134308#3808841>, @jyknight wrote:
>
>> is incorrect. Sorry to be a pain, but do you have a link that describes the issues here? I may well not be remembering properly, but I had thought previous discussions and changes were about pointer provenance issues that arise from the inttoptr direction.
>
> ptrtoint is the most basic capture <https://github.com/llvm/llvm-project/blob/4132bce9e56b00cdce8928e4ea67b136c93f46a2/llvm/lib/Analysis/CaptureTracking.cpp#L432>

Also, just see the langref description for llvm.ptrmask


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

https://reviews.llvm.org/D134308