[PATCH] D156190: [llvm-objdump] -d: don't display mapping symbols as labels

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 10:32:55 PDT 2023


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


================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:1691
         End = std::min(End, Symbols[SI].Addr);
+
       if (Start >= End || End <= StartAddress)
----------------
peter.smith wrote:
> Was this change intentional? 
Not:) Fixed. I have tried several approaches for this patch, and I forgot to drop this change after abandoning my previous approaches.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156190/new/

https://reviews.llvm.org/D156190



More information about the llvm-commits mailing list