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

Sanjay Patel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 16 13:55:37 PST 2022


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM based on the timing, results, and alternatives discussed
There's some testing in progress according to previous comments, so best to wait for that to finish in case it turns up anything new.



================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1119
+  if (IsFullLTO) {
+    FPM.addPass(SROAPass());
+  }
----------------
Is there a reason to put this down here vs. tacking it on the end of the previous IsFullLTO block? 
If LoopUnroll is the reason for adding SROA, then mention that specifically in the comment?

IIRC, all of the FullLTO predicates in this set of passes were questionable (see TODO comment above this function). They just accumulated because the code was duplicated and diverged over time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136806



More information about the cfe-commits mailing list