[Mlir-commits] [mlir] 5066551 - [mlir] Apply ClangTidy fixes (NFC)

[Adrian Kuegel]

Author: Adrian Kuegel
Date: 2023-08-04T08:44:20+02:00
New Revision: 50665511c79f62d97c0d6602e1577b0abe1b982f

URL: https://github.com/llvm/llvm-project/commit/50665511c79f62d97c0d6602e1577b0abe1b982f
DIFF: https://github.com/llvm/llvm-project/commit/50665511c79f62d97c0d6602e1577b0abe1b982f.diff

LOG: [mlir] Apply ClangTidy fixes (NFC)

Remove redundant returns at end of function.

Added: 
    

Modified: 
    mlir/test/lib/IR/TestBytecodeCallbacks.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/test/lib/IR/TestBytecodeCallbacks.cpp b/mlir/test/lib/IR/TestBytecodeCallbacks.cpp
index 4e02bdc600cd63..cc884c7cc76f37 100644
--- a/mlir/test/lib/IR/TestBytecodeCallbacks.cpp
+++ b/mlir/test/lib/IR/TestBytecodeCallbacks.cpp
@@ -108,7 +108,6 @@ struct TestBytecodeCallbackPass
     // Print the module to the output stream, so that we can filecheck the
     // result.
     newModuleOp->print(llvm::outs());
-    return;
   }
 
   // Test0: let's assume that versions older than 2.0 were relying on a special
@@ -184,7 +183,6 @@ struct TestBytecodeCallbackPass
           return success();
         });
     doRoundtripWithConfigs(op, writeConfig, parseConfig);
-    return;
   }
 
   // Test1: When writing bytecode, we override the encoding of TestI32Type with
@@ -216,7 +214,6 @@ struct TestBytecodeCallbackPass
     // We natively parse the attribute as a builtin, so no callback needed.
     ParserConfig parseConfig(op->getContext(), /*verifyAfterParse=*/true);
     doRoundtripWithConfigs(op, writeConfig, parseConfig);
-    return;
   }
 
   // Test2: When writing bytecode, we write standard builtin IntegerTypes. At
@@ -244,7 +241,6 @@ struct TestBytecodeCallbackPass
           return success();
         });
     doRoundtripWithConfigs(op, writeConfig, parseConfig);
-    return;
   }
 
   // Test3: When writing bytecode, we override the encoding of
@@ -280,7 +276,6 @@ struct TestBytecodeCallbackPass
     // We natively parse the attribute as a builtin, so no callback needed.
     ParserConfig parseConfig(op->getContext(), /*verifyAfterParse=*/false);
     doRoundtripWithConfigs(op, writeConfig, parseConfig);
-    return;
   }
 
   // Test4: When writing bytecode, we write standard builtin
@@ -318,7 +313,6 @@ struct TestBytecodeCallbackPass
           return success();
         });
     doRoundtripWithConfigs(op, writeConfig, parseConfig);
-    return;
   }
 
   // Test5: When writing bytecode, we want TestDialect to use nothing else than
@@ -360,7 +354,6 @@ struct TestBytecodeCallbackPass
           return success();
         });
     doRoundtripWithConfigs(op, writeConfig, parseConfig);
-    return;
   }
 };
 } // namespace