[PATCH] D136806: [Pipelines] Introduce SROA after (final, full) loop unrolling

[Roman Lebedev via Phabricator via llvm-commits]

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