================ @@ -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