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

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 20 11:44:16 PDT 2021


nikic added a comment.

The change you made for the NewPM doesn't just disabled MSSA -- it enables AST instead, which is even worse :) What you'd actually want for this case is an additional option that disables the memory optimizations in LICM entirely and uses neither MSSA nor AST. That should make this change much cheaper, as MSSA/AST should be the primary costs here.


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

https://reviews.llvm.org/D100802



More information about the llvm-commits mailing list