[PATCH] D124364: [Pipelines] Enable EarlyCSE after CoroCleanup to avoid runtime performance losses (5/5)

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

In D124364#3472330 <https://reviews.llvm.org/D124364#3472330>, @aeubanks wrote:

> would it make sense to put all of the coroutine lower passes right at the beginning of the pipeline? e.g. around LowerExpectIntrinsicPass? is there a reason CoroSplit is interleaved in the CGSCC pass manager?
>
> I'd like to understand the constraints better before going forward with a solution that currently seems unprincipled to me

there's also been some talk about building infrastructure to conditionally running passes if some other pass has done some transformation, but ideally we'd have a simpler solution here


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

https://reviews.llvm.org/D124364