[Mlir-commits] [mlir] 91f17ba - [mlir] Print the correct tool name in mlirTranslateMain

Andrew Young llvmlistbot at llvm.org
Tue Jan 5 19:17:12 PST 2021


Author: Andrew Young
Date: 2021-01-05T19:17:01-08:00
New Revision: 91f17ba24e86bfdd6e794e564f1e562b1d25e156

URL: https://github.com/llvm/llvm-project/commit/91f17ba24e86bfdd6e794e564f1e562b1d25e156
DIFF: https://github.com/llvm/llvm-project/commit/91f17ba24e86bfdd6e794e564f1e562b1d25e156.diff

LOG: [mlir] Print the correct tool name in mlirTranslateMain

    The passed in tool name is not used, causing the wrong tool name to be printed by the help text.

Reviewed By: mehdi_amini

Differential Revision: https://reviews.llvm.org/D94120

Added: 
    mlir/test/mlir-translate/commandline.mlir

Modified: 
    mlir/lib/Translation/Translation.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Translation/Translation.cpp b/mlir/lib/Translation/Translation.cpp
index 5c5ecb7e46c4..3c17ea237085 100644
--- a/mlir/lib/Translation/Translation.cpp
+++ b/mlir/lib/Translation/Translation.cpp
@@ -157,7 +157,7 @@ LogicalResult mlir::mlirTranslateMain(int argc, char **argv,
                            llvm::cl::Required);
   registerAsmPrinterCLOptions();
   registerMLIRContextCLOptions();
-  llvm::cl::ParseCommandLineOptions(argc, argv, "MLIR translation driver\n");
+  llvm::cl::ParseCommandLineOptions(argc, argv, toolName);
 
   std::string errorMessage;
   auto input = openInputFile(inputFilename, &errorMessage);

diff  --git a/mlir/test/mlir-translate/commandline.mlir b/mlir/test/mlir-translate/commandline.mlir
new file mode 100644
index 000000000000..e55e7346cf48
--- /dev/null
+++ b/mlir/test/mlir-translate/commandline.mlir
@@ -0,0 +1,2 @@
+// RUN: mlir-translate --help | FileCheck %s
+// CHECK: OVERVIEW: MLIR Translation Testing Tool
\ No newline at end of file


        


More information about the Mlir-commits mailing list