[PATCH] D95354: [yaml2obj] - Refine how we set the sh_link field. NFCI.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 01:48:37 PST 2021


grimar added a comment.

It turns out this wasn't NFC, one of LLDB test cases started to crash. 
I've fixed it in rG2a33b092f5b175a21680b3980ff2019bc1c65b12 <https://reviews.llvm.org/rG2a33b092f5b175a21680b3980ff2019bc1c65b12>.

The difference was that before this patch we set the `sh_link` for symbol tables basing on their names (E.g ".symtab"),
now we set this field basing on section types (e.g SHT_SYMTAB).
The mentioned test case has a `.text` section of type `SHT_SYMTAB` and it did not expect to see a sh_link field set it seeems.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95354



More information about the llvm-commits mailing list