[PATCH] D33540: [PM/ThinLTO] Port the ThinLTO pipeline (both components) to the new PM.

[Davide Italiano via Phabricator via llvm-commits]

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

Just few comments.



================
Comment at: lib/Passes/PassBuilder.cpp:751
+
+  // Force any function attributes we want the rest of the pipeline te observe.
+  MPM.addPass(ForceFunctionAttrsPass());
----------------
s/te/to/ I guess.


================
Comment at: lib/Passes/PassBuilder.cpp:995-997
+  if (Name.startswith("default") || Name.startswith("thinlto") ||
+      Name.startswith("lto"))
     return DefaultAliasRegex.match(Name);
----------------
this and the one below could be factored out in a single helper so we don't have to change two cases. It can be a follow-up.


https://reviews.llvm.org/D33540