VectorizerStart extension is module callback in old PM, but is function
callback in new PM. We lack a module extension point between end of
buildModuleSimplificationPipeline and the function optimization
(including vectorizer) pipeline. So this patch adds a new module
extension point before the function optimization pipeline.
Details
Details
- Reviewers
aeubanks - Commits
- rG0bda12b5bcea: [NewPM] Add OptimizerEarly module extension point
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/Passes/PassBuilderPipelines.cpp | ||
---|---|---|
1163 | not sure your intended use case, but perhaps putting it after RecomputeGlobalsAAPass would give these passes better AA? |
llvm/lib/Passes/PassBuilderPipelines.cpp | ||
---|---|---|
1163 | Done. Our use case is that a few module passes need to run before vectorizer. |
Comment Actions
RISCV LIT fails should not be related to this patch. Today I've verified this patch with check-llvm on latest llvm.
@aeubanks could you please review the new change? If it looks good, please submit for me, thank you.
Comment Actions
is there a "First Last <email>" you'd like to use in the commit? arc patch isn't automatically picking that up
not sure your intended use case, but perhaps putting it after RecomputeGlobalsAAPass would give these passes better AA?