[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