[PATCH] D59097: [yaml2obj]Lookup relocation symbols in dynamic symbol when .dynsym referenced

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 11 09:09:46 PDT 2019


grimar added inline comments.


================
Comment at: tools/yaml2obj/yaml2elf.cpp:868
+                              Doc.DynamicSymbols.Weak))
     return 1;
 
----------------
grimar wrote:
> This place looks a bit overcomplicated (seems requires refactoring), but it is not a problem of this patch.
I committed NFC in r355832. Open for comments.



Repository:
  rL LLVM

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

https://reviews.llvm.org/D59097





More information about the llvm-commits mailing list