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

Dhruv Chawla via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 03:43:34 PDT 2023


0xdc03 added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1281
   // alignment information, try to re-derive it here.
   FPM.addPass(AlignmentFromAssumptionsPass());
 }
----------------
aeubanks wrote:
> can this also be replaced with the new pass?
I do not believe so, because this uses SCEV information whereas the InferAlignment pass does not. It can probably be implemented in a later patch though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158600



More information about the llvm-commits mailing list