[llvm] 9299ca7 - [Coroutines] Fix inline comment about frame layout (#78626)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 18 17:46:20 PST 2024


Author: yonillasky
Date: 2024-01-19T09:46:15+08:00
New Revision: 9299ca797ae6d0c5ac008ef2126ab53f214daa21

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

LOG: [Coroutines] Fix inline comment about frame layout (#78626)

`ResumeIndex` isn't part of the frame struct header, so it necessarily
appears after the promise.

Co-authored-by: Yoni Lavi <yoni.lavi at nextsilicon.com>

Added: 
    

Modified: 
    llvm/lib/Transforms/Coroutines/CoroFrame.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Coroutines/CoroFrame.cpp b/llvm/lib/Transforms/Coroutines/CoroFrame.cpp
index 429344652a4c49..db09337215c10f 100644
--- a/llvm/lib/Transforms/Coroutines/CoroFrame.cpp
+++ b/llvm/lib/Transforms/Coroutines/CoroFrame.cpp
@@ -1289,8 +1289,8 @@ static void buildFrameDebugInfo(Function &F, coro::Shape &Shape,
 //   struct f.frame {
 //     ResumeFnTy ResumeFnAddr;
 //     ResumeFnTy DestroyFnAddr;
-//     int ResumeIndex;
 //     ... promise (if present) ...
+//     int ResumeIndex;
 //     ... spills ...
 //   };
 static StructType *buildFrameType(Function &F, coro::Shape &Shape,


        


More information about the llvm-commits mailing list