[PATCH] D105131: Change the phase ordering of SROA in the LTO to enable more cse opportunities

Tue Jun 29 10:50:12 PDT 2021

lebedev.ri added a comment.

In D105131#2847819 <https://reviews.llvm.org/D105131#2847819>, @jinlin wrote:

> In D105131#2847681 <https://reviews.llvm.org/D105131#2847681>, @lebedev.ri wrote:
>
>> This is only changing LTO side of obsolete pass manager.
>> If that is intentional, i'm not sure it's really worthwhile - it's obsolete pass manager is obsolete.
>
> When I added -flto, this pass will be invoked. The LTO is used in the release build at Uber. Why do you think it is obsolete?

Because now the default is the New Pass Manager.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105131