================ @@ -316,8 +316,13 @@ static llvm::Error parseRecord(const Record &R, unsigned ID, static llvm::Error parseRecord(const Record &R, unsigned ID, llvm::StringRef Blob, CommentInfo *I) { switch (ID) { - case COMMENT_KIND: - return decodeRecord(R, I->Kind, Blob); + case COMMENT_KIND: { ---------------- ilovepi wrote:
The braces still seem to be here. We return from this block and shouldn't have any decls that would conflict, so the braces aren't needed are they? I'm generally fine w/ bracing on switch statements, but the convention in this function/file is not to have them, so I'd like to stay consistent. https://github.com/llvm/llvm-project/pull/142273 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits