[PATCH] D139131: [llvm-objdump] Avoid using mapping symbols as branch target labels
    Simon Tatham via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Dec  5 01:20:05 PST 2022
    
    
  
simon_tatham accepted this revision.
simon_tatham added a comment.
Yes, I agree too – definitely a good change.
I especially like the part where a mapping symbol isn't used //even if it's the only symbol at that address at all//, as seen in one of the modified tests. `functionname+0x34` really is an improvement on `$x.12`!
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139131/new/
https://reviews.llvm.org/D139131
    
    
More information about the llvm-commits
mailing list