lebedev.ri added a comment.
In D136806#3926319 <https://reviews.llvm.org/D136806#3926319>, @arsenm wrote:
> I think SROA after unroll is important. It's practically the main reason to
> unroll
================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1200-1201
FPM.addPass(WarnMissedTransformationsPass());
+ FPM.addPass(SROAPass());
FPM.addPass(InstCombinePass());
FPM.addPass(
----------------
As the cross-revision diff shows, running sroa before instcombine,
non-unexpectedly, makes more sense.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136806/new/
https://reviews.llvm.org/D136806
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
