[PATCH] D53576: [llvm-objdump] Add alias option `--full-contents` for `-s` (PR39404)

Xing via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 24 09:00:05 PDT 2018


Higuoxing added a comment.

In https://reviews.llvm.org/D53576#1274371, @jhenderson wrote:

> LGTM.
>
> Do you need me to commit it?
>
> I have one inlined query about formatting. If you haven't clang-formatted the area, please do so before committing.


Thanks a lot,

The formatted one is

  cl::opt<bool>
      llvm::SectionContents("full-contents",
                            cl::desc("Display the content of each section"));
  static cl::alias SectionContentsShort("s",
                                        cl::desc("Alias for --full-contents"),
                                        cl::aliasopt(SectionContents));

but I format it as previous function `DynamicRelocations`


https://reviews.llvm.org/D53576





More information about the llvm-commits mailing list