[PATCH] D87346: [elf2yaml] Fix dumping a debug section whose name is not recognized.

Xing GUO via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 9 01:36:09 PDT 2020


Higuoxing added inline comments.


================
Comment at: llvm/tools/obj2yaml/elf2yaml.cpp:424
+                              "the parser for the %s section is not supported",
+                              RawSec->Name.str().c_str());
 
----------------
grimar wrote:
> Couldn't it be just
> 
> ```
> else
>   continue;
> ```
> 
> ?
Thanks, it looks much simpler!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87346



More information about the llvm-commits mailing list