[PATCH] D158600: [InferAlignment] Enable InferAlignment pass by default

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D158600#4649040 <https://reviews.llvm.org/D158600#4649040>, @nlopes wrote:

> I'm confused...
>
> This pass regresses alignment information in a lot of tests. I've also observed in when compiling real programs.

The test regressions in this diff are just an artifact of inference moving into a different pass (the tests run instcombine as part of the pipeline, while is no longer responsible for alignment inference). If you see end-to-end regressions in alignment information, that would be a problem though. Do you have an example?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158600