[PATCH] D35477: [ELF] - Print options aliases in --help

[Rafael Avila de Espindola via llvm-commits]

Shouldn't this include a testcase?

Cheers,
Rafael

George Rimar via Phabricator <reviews at reviews.llvm.org> writes:

> grimar updated this revision to Diff 107875.
> grimar edited the summary of this revision.
> grimar added a comment.
>
> - With currently used --help rendering approach, this one
>
> becomes trivial single line change. (still depends on https://reviews.llvm.org/D35476 though).
>
>
> https://reviews.llvm.org/D35477
>
> Files:
>   ELF/DriverUtils.cpp
>
>
> Index: ELF/DriverUtils.cpp
> ===================================================================
> --- ELF/DriverUtils.cpp
> +++ ELF/DriverUtils.cpp
> @@ -116,7 +116,8 @@
>  
>  void elf::printHelp(const char *Argv0) {
>    ELFOptTable Table;
> -  Table.PrintHelp(outs(), Argv0, "lld", false);
> +  Table.PrintHelp(outs(), Argv0, "lld", false /*ShowHidden*/,
> +                  true /*InferHelpText*/);
>    outs() << "\n";
>  
>    // Scripts generated by Libtool versions up to at least 2.4.6 (the most
>
>
> Index: ELF/DriverUtils.cpp
> ===================================================================
> --- ELF/DriverUtils.cpp
> +++ ELF/DriverUtils.cpp
> @@ -116,7 +116,8 @@
>  
>  void elf::printHelp(const char *Argv0) {
>    ELFOptTable Table;
> -  Table.PrintHelp(outs(), Argv0, "lld", false);
> +  Table.PrintHelp(outs(), Argv0, "lld", false /*ShowHidden*/,
> +                  true /*InferHelpText*/);
>    outs() << "\n";
>  
>    // Scripts generated by Libtool versions up to at least 2.4.6 (the most