[PATCH] D154010: [InstCombine] Canonicalize `getelementptr` patterns to `@llvm.ptrmask`

[Noah Goldstein via Phabricator via llvm-commits]

goldstein.w.n added a comment.

In D154010#4457648 <https://reviews.llvm.org/D154010#4457648>, @arichardson wrote:

> Would it also make sense to update the align_up codegen to use this intrinsic directly? I didn't do it at the time since there were concerns about using ptrmask, but it seems like optimization passes handle it now?

I think probably. What prompted this was the recent support added for it in knownbits + the fact that we do a better just assigning access attributes (and ingeneral tracking pointers) when there is no `ptrtoint`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154010