[llvm] eed8df6 - [Coroutines][Docs] Remove frame packing as a TODO

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


Author: Atmn Patel
Date: 2020-11-02T15:57:04-05:00
New Revision: eed8df6a131453dee779f7e3554d4e0bb591318b

URL: https://github.com/llvm/llvm-project/commit/eed8df6a131453dee779f7e3554d4e0bb591318b
DIFF: https://github.com/llvm/llvm-project/commit/eed8df6a131453dee779f7e3554d4e0bb591318b.diff

LOG: [Coroutines][Docs] Remove frame packing as a TODO

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

Differential Revision: https://reviews.llvm.org/D90550

Added: 
    

Modified: 
    llvm/docs/Coroutines.rst

Removed: 
    


################################################################################
diff  --git a/llvm/docs/Coroutines.rst b/llvm/docs/Coroutines.rst
index 5afb33fa0a0a..8c8d13a45aa6 100644
--- a/llvm/docs/Coroutines.rst
+++ b/llvm/docs/Coroutines.rst
@@ -1535,10 +1535,6 @@ earlier passes.
 
 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.


        


More information about the llvm-commits mailing list