[llvm] [llvm-objdump] Honor ELF symbol sizes in selective disassembly (PR #224068)

[James Henderson via llvm-commits]

================
@@ -2255,26 +2276,18 @@ disassembleObject(ObjectFile &Obj, const ObjectFile &DbgObj,
       // of which symbols to display by printing _all_ of them at this address
       // if the user asked for all symbols.
       //
-      // That way, '--show-all-symbols --disassemble-symbol=foo' will print
-      // only the chunk of code headed by 'foo', but also show any other
-      // symbols defined at that address, such as aliases for 'foo', or the ARM
-      // mapping symbol preceding its code.
+      // That way, '--show-all-symbols --disassemble-symbols=foo' will print
+      // only the range selected by 'foo', but also show other symbols within
+      // that range, such as aliases for 'foo' or ARM mapping symbols.
       if (ShowAllSymbols) {
----------------
jh7370 wrote:

Do we have a test case for --show-all-symbols + --disassemble-symbols with non-zero-sized symbols? Particularly aliases and "inner" symbols.

https://github.com/llvm/llvm-project/pull/224068