[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