[llvm] [Coroutines] Fix inline comment about frame layout (PR #78626)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 18 13:19:26 PST 2024


https://github.com/yonillasky created https://github.com/llvm/llvm-project/pull/78626

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


>From 23e25ccc245a15f44c14d4d271c24c5810c03b23 Mon Sep 17 00:00:00 2001
From: Yoni Lavi <yoni.lavi at nextsilicon.com>
Date: Thu, 18 Jan 2024 23:16:59 +0200
Subject: [PATCH] [Coroutines] Fix inline comment about frame layout

---
 llvm/lib/Transforms/Coroutines/CoroFrame.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/lib/Transforms/Coroutines/CoroFrame.cpp b/llvm/lib/Transforms/Coroutines/CoroFrame.cpp
index 429344652a4c492..db09337215c10f3 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