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

[Alina Sbirlea via Phabricator via cfe-commits]

asbirlea added a comment.

IIUC compile time impact for adding another SROA (the one outside LTO) is negligible?

Regarding the principle of adding another pass and where in the pipeline, we're still at a case by case basis. We had a discussion/round table at LLVM Dev on the documenting the current new pass manager pipeline precisely for understanding pass dependencies and being able to get resolutions on cases such as this. I'm still putting together the summary for that but it should be up on discourse this week.

Testing this change in parallel - I'll follow up tomorrow.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136806