[PATCH] D158529: [InferAlignment] Implement InferAlignmentPass

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

In D158529#4607972 <https://reviews.llvm.org/D158529#4607972>, @nikic wrote:

> In D158529#4607917 <https://reviews.llvm.org/D158529#4607917>, @efriedma wrote:
>
>> Alignment information is most useful in the backend, yes. But it's also relevant for alias analysis and cost modeling, so there could be some impact to delaying the computation.  We might need both an early pass and a late pass.  We might not need to do it every instcombine run, though.
>
> Hm, where does alias analysis use alignment? I don't remember seeing that anywhere.

I think I got confused here.  I'm pretty sure the backend alias analysis does use alignment, but maybe IR-level analysis doesn't.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158529