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

[Florian Hahn via Phabricator via llvm-commits]

fhahn updated this revision to Diff 350622.
fhahn added a comment.
Herald added a subscriber: ormris.

Updated to also add the `-debug-pass-manager` option, so it is easier to check that `-use-new-pm` works as expected. I also pinned the remarks tests to use the legacy PM and updated `llvm/test/ThinLTO/X86/newpm-basic.ll` to  check that the new PM is run via the debug output.

In D102627#2763674 <https://reviews.llvm.org/D102627#2763674>, @steven_wu wrote:

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

It's still not unified with the new LTO API, so if anything in the pipeline setup changes in the new LTO API we have to also update the code here. But other than that, ThinLTO should use the NewPM same as with the new API.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102627

Files:
  llvm/include/llvm/LTO/legacy/ThinLTOCodeGenerator.h
  llvm/lib/LTO/ThinLTOCodeGenerator.cpp
  llvm/test/ThinLTO/X86/diagnostic-handler-remarks-with-hotness.ll
  llvm/test/ThinLTO/X86/diagnostic-handler-remarks.ll
  llvm/test/ThinLTO/X86/newpm-basic.ll
  llvm/tools/llvm-lto/llvm-lto.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102627.350622.patch
Type: text/x-patch
Size: 9173 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210608/7c1603c4/attachment.bin>