[PATCH] D105877: [Coroutines] Run coroutine passes by default

ChuanqiXu added a comment. In D105877#2924157 , @ChuanqiXu wrote: > In D105877#2923257 , @nikic wrote: > >> I noticed that this change had a measurable impact on `O0` memory usage, >> which I wouldn't have expect

[PATCH] D105877: [Coroutines] Run coroutine passes by default

ChuanqiXu added a comment. In D105877#2923257 , @nikic wrote: > I noticed that this change had a measurable impact on `O0` memory usage, > which I wouldn't have expected > (https://llvm-compile-time-tracker.com/compare.php?from=0f9e6451a836886f39137818c

[PATCH] D105877: [Coroutines] Run coroutine passes by default

aeubanks added a comment. it's probably because we're constructing the call graph every time at -O0 we want to skip all of that if there are no coroutine intrinsics in the module. we can't really express that currently. if we'd want to do something like that, we'd need a wrapper around the pass

[PATCH] D105877: [Coroutines] Run coroutine passes by default

lxfind added a comment. In D105877#2923257 , @nikic wrote: > I noticed that this change had a measurable impact on `O0` memory usage, > which I wouldn't have expected > (https://llvm-compile-time-tracker.com/compare.php?from=0f9e6451a836886f39137818c4f0

[PATCH] D105877: [Coroutines] Run coroutine passes by default

nikic added a comment. I noticed that this change had a measurable impact on `O0` memory usage, which I wouldn't have expected (https://llvm-compile-time-tracker.com/compare.php?from=0f9e6451a836886f39137818c4f0cfd69ae31e62&to=8a1727ba51d262365b0d9fe10fef7e50da7022cd&stat=max-rss). Any idea wha

[PATCH] D105877: [Coroutines] Run coroutine passes by default

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8a1727ba51d2: [Coroutines] Run coroutine passes by default (authored by ChuanqiXu). Herald added a project: clang. Herald added a subscriber: cfe-com