This is an archive of the discontinued LLVM Phabricator instance.

[Coroutines][Docs] Remove frame packing as a TODO
ClosedPublic

Authored by atmnpatel on Nov 1 2020, 3:52 AM.

Details

Summary

This has already been done by @rjmccall in D76526 (49e5a97ec363), and
9514c048d89e. We should remove this from the docs.

Diff Detail

Event Timeline

atmnpatel created this revision.Nov 1 2020, 3:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 1 2020, 3:52 AM
atmnpatel requested review of this revision.Nov 1 2020, 3:52 AM
rjmccall accepted this revision.Nov 2 2020, 11:11 AM

I'm not sure how overlap-aware our packing is, but yeah, I'm fine with removing this.

This revision is now accepted and ready to land.Nov 2 2020, 11:11 AM
This revision was landed with ongoing or failed builds.Nov 2 2020, 12:57 PM
This revision was automatically updated to reflect the committed changes.