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

Reply via email to