[mlir][mlir-translation] patch for standalone-translation command line description missing.

[changkaiyan]

Summary

When I run standalone-translate executable file, I expect the following command option can be different from its description. However, I can not separately set the description, which means there must be some unexpected relationship of command option and command description in mlir repository. I have found the error in mlir/lib/Tools/mlir-translate/Translation.cpp as shown in Code Snippet 1. To make application programmer make use of the description, there should be some override functions to provide this customized description.

# things now (unexpected)
OVERVIEW: MLIR Translation Tool
USAGE: standalone-translate [options] <input file>

OPTIONS:

Color Options:

  --color                                              - Use colors in output (default=autodetect)

General options:

  Translation to perform
      --myoption                                 - myoption

# expected
OVERVIEW: MLIR Translation Tool
USAGE: standalone-translate [options] <input file>

OPTIONS:

Color Options:

  --color                                              - Use colors in output (default=autodetect)

General options:

  Translation to perform
      --myoption                                 - something different

// Code Snippet 1

TranslationParser::TranslationParser(llvm::cl::Option &opt)
    : llvm::cl::parser<const TranslateFunction *>(opt) {
  for (const auto &kv : getTranslationRegistry())
    addLiteralOption(kv.first(), &kv.second, kv.first()); // argument 0 is option, argument 2 is description. It should be different.
}

Test Plan:
The additional description can be tested by the standalone project.

[python3kgae]

Line ending change only?
Cannot see a real difference in the patch :(

[changkaiyan]

In D134696#3822525, @python3kgae wrote:

Line ending change only?
Cannot see a real difference in the patch :(

Sorry, I may upload the wrong diff file, I will check the right version and reupload it.

[changkaiyan]

The update diff. Fixed the previous version.

[python3kgae]

In D134696#3822622, @changkaiyan wrote:

The update diff. Fixed the previous version.

This is not a full diff. To make reviews easier, please always include as much context as possible with your diff!

To get a full diff, use one of the following commands (or just use Arcanist to upload your patch):

git show HEAD -U999999 > mypatch.patch
git diff -U999999 u > mypatch.patch
git diff HEAD~1 -U999999 > mypatch.patch

You can find information about Arcanist at https://llvm.org/docs/Phabricator.html and https://secure.phabricator.com/book/phabricator/article/arcanist_quick_start/

[changkaiyan]

Update the diff file. The format is correct in this version.

[changkaiyan]

In D134696#3822777, @python3kgae wrote:

In D134696#3822622, @changkaiyan wrote:

The update diff. Fixed the previous version.

This is not a full diff. To make reviews easier, please always include as much context as possible with your diff!

To get a full diff, use one of the following commands (or just use Arcanist to upload your patch):

git show HEAD -U999999 > mypatch.patch
git diff -U999999 u > mypatch.patch
git diff HEAD~1 -U999999 > mypatch.patch

You can find information about Arcanist at https://llvm.org/docs/Phabricator.html and https://secure.phabricator.com/book/phabricator/article/arcanist_quick_start/

Thanks for your comment. I have updated the diff file.

[rriddle]

Thanks for fixing the diff, it's much clearer what this is about now.

mlir/include/mlir/Tools/mlir-translate/Translation.h

0–1

I would just always require a description.

mlir/lib/Tools/mlir-translate/Translation.cpp

1–6

I think it would be cleaner to restructure getTranslationRegistry instead of adding a secondary map. What about introducing a proper Translation (or better name) struct that contains the TranslateFunction+arg+description?

[changkaiyan]

mlir/include/mlir/Tools/mlir-translate/Translation.h

0–1

Some legacy code can not use the new signature. One solution is for compatibility, these functions only with parameter name are retained. The second solution is to modify all the related code in mlir repository, add a description to them. I prefer the second one, because this could enhance users' experience.

Example:

/home/changkaiyan/LLVM/mlir/lib/Target/SPIRV/TranslateRegistration.cpp: In function ‘void mlir::registerTestRoundtripDebugSPIRV()’:
/home/changkaiyan/LLVM/mlir/lib/Target/SPIRV/TranslateRegistration.cpp:182:8: error: no matching function for call to ‘mlir::TranslateFromMLIRRegistration::TranslateFromMLIRRegistration(const char [27], mlir::registerTestRoundtripDebugSPIRV()::<lambda(mlir::ModuleOp, llvm::raw_ostream&)>, mlir::registerTestRoundtripDebugSPIRV()::<lambda(mlir::DialectRegistry&)>)’
       });
        ^

Related Target: LLVMIR, SPIRV, Cpp.

[changkaiyan]

Solve the related problem with this patch.

[changkaiyan]

Any problem to this version? I think this is a better patch and I am waiting for the comment before commit this.

[rriddle]

LG, thanks!

mlir/lib/Tools/mlir-translate/Translation.cpp

0–1

Can we support something like:

1–4

Can you wrap this in an anonymous namespace?

mlir/examples/standalone/standalone-translate/standalone-translate.cpp

-//===- standalone-translate.cpp ---------------------------------*- C++ -*-===//
-//
-// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-//
-// This is a command line utility that translates a file from/to MLIR using one
-// of the registered translations.
-//
-//===----------------------------------------------------------------------===//
-
-#include "mlir/InitAllTranslations.h"
-#include "mlir/Support/LogicalResult.h"
-#include "mlir/Tools/mlir-translate/MlirTranslateMain.h"
-
-#include "Standalone/StandaloneDialect.h"
-
-int main(int argc, char **argv) {
-  mlir::registerAllTranslations();
-
-  // TODO: Register standalone translations here.
-
-  return failed(
-      mlir::mlirTranslateMain(argc, argv, "MLIR Translation Testing Tool"));
-}
+//===- standalone-translate.cpp ---------------------------------*- C++ -*-===//
+//
+// This file is licensed under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+//
+// This is a command line utility that translates a file from/to MLIR using one
+// of the registered translations.
+//
+//===----------------------------------------------------------------------===//
+
+#include "mlir/InitAllTranslations.h"
+#include "mlir/Support/LogicalResult.h"
+#include "mlir/Tools/mlir-translate/MlirTranslateMain.h"
+
+#include "Standalone/StandaloneDialect.h"
+
+int main(int argc, char **argv) {
+  mlir::registerAllTranslations();
+
+  // TODO: Register standalone translations here.
+
+  return failed(
+      mlir::mlirTranslateMain(argc, argv, "MLIR Translation Testing Tool"));
+}

mlir/include/mlir/Tools/mlir-translate/Translation.h

-//===- Translation.h - Translation registry ---------------------*- C++ -*-===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-//
-// Registry for user-provided translations.
-//
-//===----------------------------------------------------------------------===//
-
-#ifndef MLIR_TOOLS_MLIRTRANSLATE_TRANSLATION_H
-#define MLIR_TOOLS_MLIRTRANSLATE_TRANSLATION_H
-
-#include "llvm/Support/CommandLine.h"
-
-namespace llvm {
-class MemoryBuffer;
-class SourceMgr;
-class StringRef;
-} // namespace llvm
-
-namespace mlir {
-class DialectRegistry;
-struct LogicalResult;
-class MLIRContext;
-class ModuleOp;
-template <typename OpTy>
-class OwningOpRef;
-
-/// Interface of the function that translates the sources managed by `sourceMgr`
-/// to MLIR. The source manager has at least one buffer. The implementation
-/// should create a new MLIR ModuleOp in the given context and return a pointer
-/// to it, or a nullptr in case of any error.
-using TranslateSourceMgrToMLIRFunction = std::function<OwningOpRef<ModuleOp>(
-    llvm::SourceMgr &sourceMgr, MLIRContext *)>;
-
-/// Interface of the function that translates the given string to MLIR. The
-/// implementation should create a new MLIR ModuleOp in the given context. If
-/// source-related error reporting is required from within the function, use
-/// TranslateSourceMgrToMLIRFunction instead.
-using TranslateStringRefToMLIRFunction =
-    std::function<OwningOpRef<ModuleOp>(llvm::StringRef, MLIRContext *)>;
-
-/// Interface of the function that translates MLIR to a different format and
-/// outputs the result to a stream. It is allowed to modify the module.
-using TranslateFromMLIRFunction =
-    std::function<LogicalResult(ModuleOp, llvm::raw_ostream &output)>;
-
-/// Interface of the function that performs file-to-file translation involving
-/// MLIR. The input file is held in the given MemoryBuffer; the output file
-/// should be written to the given raw_ostream. The implementation should create
-/// all MLIR constructs needed during the process inside the given context. This
-/// can be used for round-tripping external formats through the MLIR system.
-using TranslateFunction = std::function<LogicalResult(
-    llvm::SourceMgr &sourceMgr, llvm::raw_ostream &output, MLIRContext *)>;
-
-/// Use Translate[ToMLIR|FromMLIR]Registration as an initializer that
-/// registers a function and associates it with name. This requires that a
-/// translation has not been registered to a given name.
-///
-/// Usage:
-///
-///   // At file scope.
-///   namespace mlir {
-///   void registerTRexToMLIRRegistration() {
-///     TranslateToMLIRRegistration Unused(&MySubCommand, [] { ... });
-///   }
-///   } // namespace mlir
-///
-/// \{
-struct TranslateToMLIRRegistration {
-  TranslateToMLIRRegistration(llvm::StringRef name,
-                              const TranslateSourceMgrToMLIRFunction &function);
-  TranslateToMLIRRegistration(llvm::StringRef name,
-                              const TranslateStringRefToMLIRFunction &function);
-};
-
-struct TranslateFromMLIRRegistration {
-  TranslateFromMLIRRegistration(
-      llvm::StringRef name, const TranslateFromMLIRFunction &function,
-      const std::function<void(DialectRegistry &)> &dialectRegistration =
-          [](DialectRegistry &) {});
-};
-struct TranslateRegistration {
-  TranslateRegistration(llvm::StringRef name,
-                        const TranslateFunction &function);
-};
-/// \}
-
-/// A command line parser for translation functions.
-struct TranslationParser : public llvm::cl::parser<const TranslateFunction *> {
-  TranslationParser(llvm::cl::Option &opt);
-
-  void printOptionInfo(const llvm::cl::Option &o,
-                       size_t globalWidth) const override;
-};
-
-} // namespace mlir
-
-#endif // MLIR_TOOLS_MLIRTRANSLATE_TRANSLATION_H
+//===- Translation.h - Translation registry ---------------------*- C++ -*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+//
+// Registry for user-provided translations.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef MLIR_TOOLS_MLIRTRANSLATE_TRANSLATION_H
+#define MLIR_TOOLS_MLIRTRANSLATE_TRANSLATION_H
+
+#include "llvm/Support/CommandLine.h"
+
+namespace llvm {
+class MemoryBuffer;
+class SourceMgr;
+class StringRef;
+} // namespace llvm
+
+namespace mlir {
+class DialectRegistry;
+struct LogicalResult;
+class MLIRContext;
+class ModuleOp;
+template <typename OpTy>
+class OwningOpRef;
+
+/// Interface of the function that translates the sources managed by `sourceMgr`
+/// to MLIR. The source manager has at least one buffer. The implementation
+/// should create a new MLIR ModuleOp in the given context and return a pointer
+/// to it, or a nullptr in case of any error.
+using TranslateSourceMgrToMLIRFunction = std::function<OwningOpRef<ModuleOp>(
+    llvm::SourceMgr &sourceMgr, MLIRContext *)>;
+
+/// Interface of the function that translates the given string to MLIR. The
+/// implementation should create a new MLIR ModuleOp in the given context. If
+/// source-related error reporting is required from within the function, use
+/// TranslateSourceMgrToMLIRFunction instead.
+using TranslateStringRefToMLIRFunction =
+    std::function<OwningOpRef<ModuleOp>(llvm::StringRef, MLIRContext *)>;
+
+/// Interface of the function that translates MLIR to a different format and
+/// outputs the result to a stream. It is allowed to modify the module.
+using TranslateFromMLIRFunction =
+    std::function<LogicalResult(ModuleOp, llvm::raw_ostream &output)>;
+
+/// Interface of the function that performs file-to-file translation involving
+/// MLIR. The input file is held in the given MemoryBuffer; the output file
+/// should be written to the given raw_ostream. The implementation should create
+/// all MLIR constructs needed during the process inside the given context. This
+/// can be used for round-tripping external formats through the MLIR system.
+using TranslateFunction = std::function<LogicalResult(
+    llvm::SourceMgr &sourceMgr, llvm::raw_ostream &output, MLIRContext *)>;
+
+/// Use Translate[ToMLIR|FromMLIR]Registration as an initializer that
+/// registers a function and associates it with name. This requires that a
+/// translation has not been registered to a given name.
+///
+/// Usage:
+///
+///   // At file scope.
+///   namespace mlir {
+///   void registerTRexToMLIRRegistration() {
+///     TranslateToMLIRRegistration Unused(&MySubCommand, [] { ... });
+///   }
+///   } // namespace mlir
+///
+/// \{
+struct TranslateToMLIRRegistration {
+  TranslateToMLIRRegistration(llvm::StringRef name,
+                              const TranslateSourceMgrToMLIRFunction &function);
+  TranslateToMLIRRegistration(llvm::StringRef name,
+                              const TranslateStringRefToMLIRFunction &function);
+};
+
+struct TranslateFromMLIRRegistration {
+  TranslateFromMLIRRegistration(
+      llvm::StringRef name, const TranslateFromMLIRFunction &function,
+      const std::function<void(DialectRegistry &)> &dialectRegistration =
+          [](DialectRegistry &) {});
+};
+struct TranslateRegistration {
+  TranslateRegistration(llvm::StringRef name,
+                        const TranslateFunction &function);
+};
+/// \}
+
+/// A command line parser for translation functions.
+struct TranslationParser : public llvm::cl::parser<const TranslateFunction *> {
+  TranslationParser(llvm::cl::Option &opt);
+
+  void printOptionInfo(const llvm::cl::Option &o,
+                       size_t globalWidth) const override;
+};
+
+} // namespace mlir
+
+#endif // MLIR_TOOLS_MLIRTRANSLATE_TRANSLATION_H

mlir/lib/Tools/mlir-translate/Translation.cpp

-//===- Translation.cpp - Translation registry -----------------------------===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-//
-// Definitions of the translation registry.
-//
-//===----------------------------------------------------------------------===//
-
-#include "mlir/Tools/mlir-translate/Translation.h"
-#include "mlir/IR/AsmState.h"
-#include "mlir/IR/BuiltinOps.h"
-#include "mlir/IR/Dialect.h"
-#include "mlir/IR/Verifier.h"
-#include "mlir/Parser/Parser.h"
-#include "llvm/Support/SourceMgr.h"
-
-using namespace mlir;
-
-//===----------------------------------------------------------------------===//
-// Translation Registry
-//===----------------------------------------------------------------------===//
-
-/// Get the mutable static map between registered file-to-file MLIR translations
-/// and the TranslateFunctions that perform those translations.
-static llvm::StringMap<TranslateFunction> &getTranslationRegistry() {
-  static llvm::StringMap<TranslateFunction> translationRegistry;
-  return translationRegistry;
-}
-
-/// Register the given translation.
-static void registerTranslation(StringRef name,
-                                const TranslateFunction &function) {
-  auto &translationRegistry = getTranslationRegistry();
-  if (translationRegistry.find(name) != translationRegistry.end())
-    llvm::report_fatal_error(
-        "Attempting to overwrite an existing <file-to-file> function");
-  assert(function &&
-         "Attempting to register an empty translate <file-to-file> function");
-  translationRegistry[name] = function;
-}
-
-TranslateRegistration::TranslateRegistration(
-    StringRef name, const TranslateFunction &function) {
-  registerTranslation(name, function);
-}
-
-//===----------------------------------------------------------------------===//
-// Translation to MLIR
-//===----------------------------------------------------------------------===//
-
-// Puts `function` into the to-MLIR translation registry unless there is already
-// a function registered for the same name.
-static void registerTranslateToMLIRFunction(
-    StringRef name, const TranslateSourceMgrToMLIRFunction &function) {
-  auto wrappedFn = [function](llvm::SourceMgr &sourceMgr, raw_ostream &output,
-                              MLIRContext *context) {
-    OwningOpRef<ModuleOp> module = function(sourceMgr, context);
-    if (!module || failed(verify(*module)))
-      return failure();
-    module->print(output);
-    return success();
-  };
-  registerTranslation(name, wrappedFn);
-}
-
-TranslateToMLIRRegistration::TranslateToMLIRRegistration(
-    StringRef name, const TranslateSourceMgrToMLIRFunction &function) {
-  registerTranslateToMLIRFunction(name, function);
-}
-
-/// Wraps `function` with a lambda that extracts a StringRef from a source
-/// manager and registers the wrapper lambda as a to-MLIR conversion.
-TranslateToMLIRRegistration::TranslateToMLIRRegistration(
-    StringRef name, const TranslateStringRefToMLIRFunction &function) {
-  registerTranslateToMLIRFunction(
-      name, [function](llvm::SourceMgr &sourceMgr, MLIRContext *ctx) {
-        const llvm::MemoryBuffer *buffer =
-            sourceMgr.getMemoryBuffer(sourceMgr.getMainFileID());
-        return function(buffer->getBuffer(), ctx);
-      });
-}
-
-//===----------------------------------------------------------------------===//
-// Translation from MLIR
-//===----------------------------------------------------------------------===//
-
-TranslateFromMLIRRegistration::TranslateFromMLIRRegistration(
-    StringRef name, const TranslateFromMLIRFunction &function,
-    const std::function<void(DialectRegistry &)> &dialectRegistration) {
-  registerTranslation(name, [function, dialectRegistration](
-                                llvm::SourceMgr &sourceMgr, raw_ostream &output,
-                                MLIRContext *context) {
-    DialectRegistry registry;
-    dialectRegistration(registry);
-    context->appendDialectRegistry(registry);
-    auto module = parseSourceFile<ModuleOp>(sourceMgr, context);
-    if (!module || failed(verify(*module)))
-      return failure();
-    return function(module.get(), output);
-  });
-}
-
-//===----------------------------------------------------------------------===//
-// Translation Parser
-//===----------------------------------------------------------------------===//
-
-TranslationParser::TranslationParser(llvm::cl::Option &opt)
-    : llvm::cl::parser<const TranslateFunction *>(opt) {
-  for (const auto &kv : getTranslationRegistry())
-    addLiteralOption(kv.first(), &kv.second, kv.first());
-}
-
-void TranslationParser::printOptionInfo(const llvm::cl::Option &o,
-                                        size_t globalWidth) const {
-  TranslationParser *tp = const_cast<TranslationParser *>(this);
-  llvm::array_pod_sort(tp->Values.begin(), tp->Values.end(),
-                       [](const TranslationParser::OptionInfo *lhs,
-                          const TranslationParser::OptionInfo *rhs) {
-                         return lhs->Name.compare(rhs->Name);
-                       });
-  llvm::cl::parser<const TranslateFunction *>::printOptionInfo(o, globalWidth);
-}
+//===- Translation.cpp - Translation registry -----------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+//
+// Definitions of the translation registry.
+//
+//===----------------------------------------------------------------------===//
+
+#include "mlir/Tools/mlir-translate/Translation.h"
+#include "mlir/IR/AsmState.h"
+#include "mlir/IR/BuiltinOps.h"
+#include "mlir/IR/Dialect.h"
+#include "mlir/IR/Verifier.h"
+#include "mlir/Parser/Parser.h"
+#include "llvm/Support/SourceMgr.h"
+
+using namespace mlir;
+
+//===----------------------------------------------------------------------===//
+// Translation Registry
+//===----------------------------------------------------------------------===//
+
+/// Get the mutable static map between registered file-to-file MLIR translations
+/// and the TranslateFunctions that perform those translations.
+static llvm::StringMap<TranslateFunction> &getTranslationRegistry() {
+  static llvm::StringMap<TranslateFunction> translationRegistry;
+  return translationRegistry;
+}
+
+/// Register the given translation.
+static void registerTranslation(StringRef name,
+                                const TranslateFunction &function) {
+  auto &translationRegistry = getTranslationRegistry();
+  if (translationRegistry.find(name) != translationRegistry.end())
+    llvm::report_fatal_error(
+        "Attempting to overwrite an existing <file-to-file> function");
+  assert(function &&
+         "Attempting to register an empty translate <file-to-file> function");
+  translationRegistry[name] = function;
+}
+
+TranslateRegistration::TranslateRegistration(
+    StringRef name, const TranslateFunction &function) {
+  registerTranslation(name, function);
+}
+
+//===----------------------------------------------------------------------===//
+// Translation to MLIR
+//===----------------------------------------------------------------------===//
+
+// Puts `function` into the to-MLIR translation registry unless there is already
+// a function registered for the same name.
+static void registerTranslateToMLIRFunction(
+    StringRef name, const TranslateSourceMgrToMLIRFunction &function) {
+  auto wrappedFn = [function](llvm::SourceMgr &sourceMgr, raw_ostream &output,
+                              MLIRContext *context) {
+    OwningOpRef<ModuleOp> module = function(sourceMgr, context);
+    if (!module || failed(verify(*module)))
+      return failure();
+    module->print(output);
+    return success();
+  };
+  registerTranslation(name, wrappedFn);
+}
+
+TranslateToMLIRRegistration::TranslateToMLIRRegistration(
+    StringRef name, const TranslateSourceMgrToMLIRFunction &function) {
+  registerTranslateToMLIRFunction(name, function);
+}
+
+/// Wraps `function` with a lambda that extracts a StringRef from a source
+/// manager and registers the wrapper lambda as a to-MLIR conversion.
+TranslateToMLIRRegistration::TranslateToMLIRRegistration(
+    StringRef name, const TranslateStringRefToMLIRFunction &function) {
+  registerTranslateToMLIRFunction(
+      name, [function](llvm::SourceMgr &sourceMgr, MLIRContext *ctx) {
+        const llvm::MemoryBuffer *buffer =
+            sourceMgr.getMemoryBuffer(sourceMgr.getMainFileID());
+        return function(buffer->getBuffer(), ctx);
+      });
+}
+
+//===----------------------------------------------------------------------===//
+// Translation from MLIR
+//===----------------------------------------------------------------------===//
+
+TranslateFromMLIRRegistration::TranslateFromMLIRRegistration(
+    StringRef name, const TranslateFromMLIRFunction &function,
+    const std::function<void(DialectRegistry &)> &dialectRegistration) {
+  registerTranslation(name, [function, dialectRegistration](
+                                llvm::SourceMgr &sourceMgr, raw_ostream &output,
+                                MLIRContext *context) {
+    DialectRegistry registry;
+    dialectRegistration(registry);
+    context->appendDialectRegistry(registry);
+    auto module = parseSourceFile<ModuleOp>(sourceMgr, context);
+    if (!module || failed(verify(*module)))
+      return failure();
+    return function(module.get(), output);
+  });
+}
+
+//===----------------------------------------------------------------------===//
+// Translation Parser
+//===----------------------------------------------------------------------===//
+
+TranslationParser::TranslationParser(llvm::cl::Option &opt)
+    : llvm::cl::parser<const TranslateFunction *>(opt) {
+  for (const auto &kv : getTranslationRegistry())
+    addLiteralOption(kv.first(), &kv.second, kv.first());
+}
+
+void TranslationParser::printOptionInfo(const llvm::cl::Option &o,
+                                        size_t globalWidth) const {
+  TranslationParser *tp = const_cast<TranslationParser *>(this);
+  llvm::array_pod_sort(tp->Values.begin(), tp->Values.end(),
+                       [](const TranslationParser::OptionInfo *lhs,
+                          const TranslationParser::OptionInfo *rhs) {
+                         return lhs->Name.compare(rhs->Name);
+                       });
+  llvm::cl::parser<const TranslateFunction *>::printOptionInfo(o, globalWidth);
+}