[PATCH] D33701: [coroutines] Call initializePass in coroutine pass constructors

Gor Nishanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 16:50:13 PDT 2017


GorNishanov added a comment.

In https://reviews.llvm.org/D33701#768335, @davide wrote:

> I thought there was a way to test skipFunction() ? See Andrew Kaylor's first series of `opt-bisect-limit=`/opt-bisection support.


Hmm... Reading more on opt-bisect-limit, I think it is incorrect to use skipFunction in coroutine passes. Coroutine passes can not be safely skipped while still allowing correct code generation. I am taking skipFunction checks out


https://reviews.llvm.org/D33701





More information about the llvm-commits mailing list