[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