[PATCH] D123925: [Pipelines] Hoist CoroEarly as a module pass

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 18 19:08:26 PDT 2022


ChuanqiXu added a comment.

In D123925#3457008 <https://reviews.llvm.org/D123925#3457008>, @aeubanks wrote:

> is the fewer calls to `declaresCoroEarlyIntrinsics` the only reason for this change? have you measured that the calls actually take up any noticeable amount of compile time?

Oh, there are other reasons to do this change. In fact, this is a part of patch that I am working on. And I feel this part is independent and looks good to me as a separate patch. So I send it here. I didn't measure the time. But I guess it wouldn't reduce a noticeable amount of compile time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123925



More information about the llvm-commits mailing list