Page MenuHomePhabricator

[coroutines] Add emission of initial and final suspends
ClosedPublic

Authored by GorNishanov on Apr 3 2017, 10:37 AM.

Details

Reviewers
EricWF
rsmith

Diff Detail

Event Timeline

Added a CHECK that for a final suspend we correctly set "IsFinalSuspend" argument of @llvm.coro.suspend

Pretty please. Gentle ping.

Weakly (sp) ping

@EricWF: Can you take a quick look? Just a sanity check. The change is rather trivial.

rsmith accepted this revision.May 22 2017, 12:06 PM
rsmith added a subscriber: rsmith.

LGTM

This revision is now accepted and ready to land.May 22 2017, 12:06 PM
GorNishanov closed this revision.May 22 2017, 10:05 PM

r303603 = 12cdab350043e7ff931af5f13d3e3afe85b76628

Thank you! The fix is incoming in a minute.

Fixed:

r303714 = 8832327ab89f3668378d70d1c4e5a218446ce36e