Tony, This patch generally looks good to me -- it clearly is an incremental improvement. One of the libiberty maintainers, such as Ian, needs to approve the patch.
https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01181.html + if (strcmp (name, ".text") == 0) + textptr = scnptr; The above code does not seem very robust. What if the application is compiled with -ffunction-sections so the text section is not named ".text"? + if (strtab == NULL) + { + XDELETEVEC (symtab); + XDELETEVEC (scnbuf); + return errmsg; The first XDELETEVEC (symtab) is indented incorrectly and should be fixed. Thanks, David
