================
@@ -688,9 +688,9 @@ static DIType *solveDIType(DIBuilder &Builder, Type *Ty,
 static void buildFrameDebugInfo(Function &F, coro::Shape &Shape,
                                 FrameDataInfo &FrameData) {
   DISubprogram *DIS = F.getSubprogram();
-  // If there is no DISubprogram for F, it implies the Function are not 
compiled
-  // with debug info. So we also don't need to generate debug info for the 
frame
-  // neither.
+  // If there is no DISubprogram for F, it implies the function is compiled
+  // without debug info. So we also don't generate debug info for the frame,
+  // either.
----------------
pogo59 wrote:

```suggestion
  // without debug info. So we also don't generate debug info for the frame.
```
The "also don't" and "either" are redundant.
Arguably the comment rewording belongs in its own patch, especially as there 
are no other changes to this file, but I'm fine with including it, especially 
as it matches another comment elsewhere.

https://github.com/llvm/llvm-project/pull/141937
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to