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

[Higuoxing]

Summary

If the debug section's name isn't recognized, it should be
dumped as a raw content section.

[grimar]

llvm/test/tools/obj2yaml/ELF/DWARF/unrecognized-debug-section.yaml

2

by obj2yaml?

llvm/tools/obj2yaml/elf2yaml.cpp

423

Couldn't it be just

else
  continue;

?

[Higuoxing]

Address review comments.

Thanks!

llvm/tools/obj2yaml/elf2yaml.cpp

423

Thanks, it looks much simpler!

[grimar]

LGTM

[jhenderson]

llvm/test/tools/obj2yaml/ELF/DWARF/unrecognized-debug-section.yaml

2

4

No need for the --check-prefix. Just use CHECK below.

[Higuoxing]

Address review comments.

Thanks for reviewing!

[jhenderson]

LGTM.

llvm/test/tools/obj2yaml/ELF/DWARF/unrecognized-debug-section.yaml

+## Test dumping a debug section when its name is not recognized by obj2yaml.
+
+# RUN: yaml2obj %s | obj2yaml | FileCheck %s
+
+#      CHECK: Sections:
+# CHECK-NEXT:   - Name:    .debug_foo
+# CHECK-NEXT:     Type:    SHT_PROGBITS
+# CHECK-NEXT:     Content: '01020304'
+# CHECK-NEXT: ...
+
+--- !ELF
+FileHeader:
+  Class: ELFCLASS64
+  Data:  ELFDATA2LSB
+  Type:  ET_EXEC
+Sections:
+  - Name:    .debug_foo
+    Type:    SHT_PROGBITS
+    Content: '01020304'

llvm/tools/obj2yaml/elf2yaml.cpp

if (ELFYAML::RawContentSection *RawSec =
             dyn_cast<ELFYAML::RawContentSection>(C.get())) {
       Error Err = Error::success();
       cantFail(std::move(Err));
 
       if (RawSec->Name == ".debug_aranges")
         Err = dumpDebugARanges(*DWARFCtx.get(), DWARF);
       else if (RawSec->Name == ".debug_str")
         Err = dumpDebugStrings(*DWARFCtx.get(), DWARF);
+      else
+        continue;
 
       // If the DWARF section cannot be successfully parsed, emit raw content
       // instead of an entry in the DWARF section of the YAML.
       if (Err)
         consumeError(std::move(Err));
       else
         RawSec->Content.reset();
     }
   }
 
   if (DWARF.getNonEmptySectionNames().empty())