[PATCH] D146534: [llvm-nm] Print EC symbol map.
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 23 14:35:28 PDT 2023
efriedma accepted this revision.
efriedma added a comment.
There's only one tool in existence that produces archives with an ecsymbols section; with your patch to llvm, there will be two. Corrupted sections aren't actually a thing anyone is likely to run into, outside of random data corruption. Certainly not as part of a normal build process.
But I guess delaying the error check until we actually try to query the table doesn't really do any harm. Sure, this is fine.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146534/new/
https://reviews.llvm.org/D146534
More information about the llvm-commits
mailing list