[PATCH] D131589: [llvm-objdump] Handle multiple syms at same addr in disassembly.
Rafael Auler via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 10 14:43:13 PDT 2022
rafauler added a comment.
Thanks Simon, I think this patch makes a nice improvement to the behavior of objdump. I'm not the code owner of these disassembly tools, but the patch looks good to me. I would rather wait for the input of whoever is more familiar with objdump, though.
================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:1494-1497
+ {
+ while (SI != SE && Symbols[SI].Addr == Start)
+ SI++;
+ SymbolsHere = ArrayRef<SymbolInfoTy>(&Symbols[FirstSI], SI - FirstSI);
----------------
Curious about the style here where we define a scope to isolate a piece of computation. I don't think I've seen this in other LLVM files or in the coding style, so I'm curious what other people think about it.
I do think it makes the code more clear to understand here, because it's only 3 lines. But on the other hand, it increases nesting on line 1509, which can actually make code harder to read (arguably).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131589/new/
https://reviews.llvm.org/D131589
More information about the llvm-commits
mailing list