[PATCH] D71542: [coroutines][PR41909] don't build dependent coroutine statements if the coroutine still has a dependent promise type

2019-12-18 Thread Philippe Daouadi via Phabricator via cfe-commits
blastrock abandoned this revision. blastrock added a comment. Indeed, I'm closing this then, I haven't tested your patch yet but the fix seems to be the exact same. You seem to have a better idea of what you are doing and your patch reflects that ^^ Thanks for the fix! Repository: rG LLVM G

[PATCH] D71542: [coroutines][PR41909] don't build dependent coroutine statements if the coroutine still has a dependent promise type

2019-12-18 Thread Brian Gesiak via Phabricator via cfe-commits
modocache requested changes to this revision. modocache added a comment. This revision now requires changes to proceed. Great minds think alike! This looks like the patch I sent in November, D70579 . I only just committed it two days ago in rG376cf43

[PATCH] D71542: [coroutines][PR41909] don't build dependent coroutine statements if the coroutine still has a dependent promise type

2019-12-16 Thread Philippe Daouadi via Phabricator via cfe-commits
blastrock created this revision. blastrock added reviewers: GorNishanov, EricWF, lewissbaker, tks2103, modocache, rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. blastrock edited the summary of this revision. This started with bug https://bugs.llvm.org/show_bug.cgi?