[PATCH] D28996: [PM] Flesh out the new pm LTO pipeline

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

I would suggest basic testing here similar to what we do for O[1-3sz] in test/Other/new-pm-defaults.ll. I'd keep the pre-link-LTO and non-LTO in that file and add a new file (new-pm-lto-defaults.ll) for the post-link LTO pipeline.



================
Comment at: lib/Passes/PassBuilder.cpp:672
+#if 0
+  MainFPM.add(AlignmentFromassumptionsPass());
+#endif
----------------
Typo here: assumptions -> Assumptions


Repository:
  rL LLVM

https://reviews.llvm.org/D28996