[PATCH] D74205: [llvm-dwarfdump] Add the --show-sections-sizes option

Sourabh Singh Tomar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 6 10:29:01 PST 2020


SouraVX added inline comments.


================
Comment at: llvm/tools/llvm-dwarfdump/SectionSizes.cpp:79
+    StringRef SectionName;
+    if (Expected<StringRef> NameOrErr = Section.getName()) {
+      SectionName = *NameOrErr;
----------------
nit: Since these are one liners. Can remove these braces from both if-else block?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74205/new/

https://reviews.llvm.org/D74205





More information about the llvm-commits mailing list