[PATCH] D100802: [PassManager] add late LICM to LTO pipeline to undo InstCombine sinking

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 338832.
spatel marked 2 inline comments as done.
spatel added a comment.

Patch updated:
Add explicit late LoopPassManager for LICM and disable MemorySSA usage from LICM pass on NewPM.

Hopefully, that matches what was suggested in the inline comment. Let me know if not. I should learn more about how the new PM works... 
It's not clear to me what the analogous change would look like for OldPM - set the Cap options to zero?


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

https://reviews.llvm.org/D100802

Files:
  llvm/lib/Passes/PassBuilder.cpp
  llvm/lib/Transforms/IPO/PassManagerBuilder.cpp
  llvm/test/Other/new-pm-lto-defaults.ll
  llvm/test/Other/opt-LTO-pipeline.ll
  llvm/test/Transforms/PhaseOrdering/lto-licm.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100802.338832.patch
Type: text/x-patch
Size: 4920 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210420/613a9a30/attachment.bin>