[llvm] [LTO][Pipelines][Coro] Handle coroutines in LTO pipeline (PR #126168)

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 10 10:41:55 PST 2025


vitalybuka wrote:

> we should be consistent between Full and ThinLTO and also skip coroutine passes in prelink FullLTO so we don't run coroutine passes twice with FullLTO

Good idea.
However, the patch as-is should not regress anyone, because they must crash already if reach this cases. I don't have good understanding what is good order of those passes. Probably we can just follow thinlto case.

Still to avoid regressions, I propose to land this patch as-is. It's essentially NFC for all who don't see the crash.
And address your suggestion in followup patch, so if it causes regression, we can easily revert it without reintroducing this crash.


https://github.com/llvm/llvm-project/pull/126168


More information about the llvm-commits mailing list