[PATCH] D158529: [InferAlignment] Implement InferAlignmentPass

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

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.

Do we want the late run to be after unrolling?

A pointer-specific computePointerAlign() might be faster to compute than computeKnownBits(); I mean, the basic logic is the same, but the KnownBits datastructure is a lot more complicated than an "int".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158529