================
@@ -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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits