[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