[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