[PATCH] D53690: [llvm-objdump] Don't Crash When Using `-a` on Non-Archives (PR39402)

[Xing via Phabricator via llvm-commits]

Higuoxing marked an inline comment as done.
Higuoxing added inline comments.


================
Comment at: tools/llvm-objdump/llvm-objdump.cpp:2309
 
   if (ArchiveHeaders && !MachOOpt)
+    printArchiveChild(ArchiveName, c);
----------------
eush wrote:
> It seems like you could just add `&& c` here and in the second overload and the signature of `printArchiveChild` could stay the same.
Thanks a lot, but I think checking `nullptr` inside `printArchiveChild` will be more readable ?


https://reviews.llvm.org/D53690