[PATCH] D102627: [LTO] Support new PM in ThinLTOCodeGenerator.

[Steven Wu via Phabricator via llvm-commits]

steven_wu added a comment.

That is easier than I think. Is there still any missing function in terms of NewPM after this patch?



================
Comment at: llvm/lib/LTO/ThinLTOCodeGenerator.cpp:528
+  else
+    optimizeModule(TheModule, TM, OptLevel, Freestanding, &Index);
 
----------------
There is another instance of `optimizeModule`. Maybe update that as well?


================
Comment at: llvm/test/ThinLTO/X86/diagnostic-handler-remarks.ll:23
 ; YAML1-NEXT: Name:            Inlined
-; YAML1-NEXT: Function:        main
+; YAML1-NEXT: Function:        foo
 ; YAML1-NEXT: Args:
----------------
Why this test case needs to be changed for switch PM?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102627