[PATCH] D99121: [IR][InstCombine] IntToPtr Produces Typeless Pointer To Byte

[Ruiling, Song via Phabricator via cfe-commits]

ruiling added a comment.

>> Sounds we still need a long way to get there? Do we think the patch acceptable as a short-term solution?
>
> Guess it's a tradeoff. Doesn't look like a load of code/complexity. - but equally, does it provide much value? Or is it enough to know it can eventually be addressed in a more general manner?

I don't have an answer for the second question, but this change would make later pass like `ScalarEvolution` can know the two pointers from the same integer by `inttoptr` are identical. Then we can merge possible consecutive memory access based on such pointers, we observed about 2% overall performance improvement for a critical workload.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99121