[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