[PATCH] D90550: [Coroutines][Docs] Remove frame packing as a TODO

Atmn Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 2 12:57:34 PST 2020


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGeed8df6a1314: [Coroutines][Docs] Remove frame packing as a TODO (authored by atmnpatel).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90550/new/

https://reviews.llvm.org/D90550

Files:
  llvm/docs/Coroutines.rst


Index: llvm/docs/Coroutines.rst
===================================================================
--- llvm/docs/Coroutines.rst
+++ llvm/docs/Coroutines.rst
@@ -1535,10 +1535,6 @@
 
 Areas Requiring Attention
 =========================
-#. A coroutine frame is bigger than it could be. Adding stack packing and stack 
-   coloring like optimization on the coroutine frame will result in tighter
-   coroutine frames.
-
 #. Take advantage of the lifetime intrinsics for the data that goes into the
    coroutine frame. Leave lifetime intrinsics as is for the data that stays in
    allocas.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90550.302379.patch
Type: text/x-patch
Size: 591 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201102/0e8fba44/attachment.bin>


More information about the llvm-commits mailing list