[PATCH] D71394: llvm-objdump should ignore Mach-O stab symbols for disassembly.
    Fangrui Song via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Dec 12 10:14:42 PST 2019
    
    
  
MaskRay added inline comments.
================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:1152
 
+    // Don't ask a MachO STAB for its section unless you know that STAB has a
+    // section, because a STAB section might not actually be a section. 
----------------
I think that this comment could be clearer.
Does this comment suggest that the reader should skip this piece of code if they don't know what a STAB section is? (I don't know what a STAB section is.)
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71394/new/
https://reviews.llvm.org/D71394
    
    
More information about the llvm-commits
mailing list