[PATCH] D94486: [LTO] Expose opt() in LTOBackend (NFC).

Steven Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 10:14:33 PST 2021


steven_wu added a comment.

LGTM with a small comment.



================
Comment at: llvm/lib/LTO/LTOBackend.cpp:486
 
+bool lto::opt(const Config &Conf, TargetMachine *TM, unsigned Task, Module &Mod,
+              bool IsThinLTO, ModuleSummaryIndex *ExportSummary,
----------------
Any reason why the function need to be moved instead of just renamed? I will reduce the diff if possible.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94486



More information about the llvm-commits mailing list