[all-commits] [llvm/llvm-project] 7d40f5: [Pipelines] Hoist CoroCleanup to avoid blocking op...

Chuanqi Xu via All-commits all-commits at lists.llvm.org
Thu May 5 00:14:31 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7d40f562e79b799a57799514693d0a73aab5d7f6
      https://github.com/llvm/llvm-project/commit/7d40f562e79b799a57799514693d0a73aab5d7f6
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2022-05-05 (Thu, 05 May 2022)

  Changed paths:
    M llvm/lib/Passes/PassBuilderPipelines.cpp
    M llvm/test/Other/new-pm-defaults.ll
    M llvm/test/Other/new-pm-thinlto-defaults.ll
    M llvm/test/Other/new-pm-thinlto-postlink-pgo-defaults.ll
    M llvm/test/Other/new-pm-thinlto-postlink-samplepgo-defaults.ll
    M llvm/test/Other/new-pm-thinlto-prelink-pgo-defaults.ll
    M llvm/test/Other/new-pm-thinlto-prelink-samplepgo-defaults.ll
    M llvm/test/Transforms/Coroutines/coro-retcon-opaque-ptr.ll
    M llvm/test/Transforms/Coroutines/coro-retcon-resume-values.ll
    M llvm/test/Transforms/Coroutines/coro-retcon-value.ll
    M llvm/test/Transforms/Coroutines/coro-retcon.ll

  Log Message:
  -----------
  [Pipelines] Hoist CoroCleanup to avoid blocking optimizations

CoroCleanup is designed to lowering all the remaining coroutine
intrinsics. It is required to run after CoroSplit only. However, the
position of CoroCleanup now is far too late. The downside here is that
the unlowered coroutine instrincs might blocking other optimizations
too. So it should be a pure win to hoist the position of CoroCleanup.

Reviewed By: aeubanks

Differential Revision: https://reviews.llvm.org/D124360




More information about the All-commits mailing list