[llvm-readobj] Prefer ILT to IAT for reading COFF imports

[smeenai]

Summary

We're seeing binutils ld produce binaries where the import address
table's NameRVA entry is actually a VA instead (i.e. it's already base
relocated), which llvm-readobj then chokes on. Both dumpbin and the
Windows loader are able to handle these binaries correctly, however, and
we can make llvm-readobj handle them correctly too by iterating the
import lookup table (which doesn't have a relocated NameRVA) rather than
the import address table.

The import lookup table and the import address table are supposed to be
identical on disk, and prior to r277298 the import lookup table would be
used by llvm-readobj -coff-imports anyway, so this shouldn't have any
functional change (except in the case of our malformed binaries). The
import lookup table can apparently be missing when using old Borland
linkers, so fall back to the import address table in that case.

Resolves PR31766.

[compnerd]

tools/llvm-readobj/COFFDumper.cpp

1442

ILT would be better I think than ImportLookupTable

1445

IAT would be better than ImportAddressTable I think

[smeenai]

tools/llvm-readobj/COFFDumper.cpp

1442

Will change before committing.

1445

Will change before committing.

tools/llvm-readobj/COFFDumper.cpp

 
 void COFFDumper::printCOFFImports() {
   // Regular imports
   for (const ImportDirectoryEntryRef &I : Obj->import_directories()) {
     DictScope Import(W, "Import");
     StringRef Name;
     error(I.getName(Name));
     W.printString("Name", Name);
-    uint32_t Addr;
-    error(I.getImportLookupTableRVA(Addr));
-    W.printHex("ImportLookupTableRVA", Addr);
-    error(I.getImportAddressTableRVA(Addr));
-    W.printHex("ImportAddressTableRVA", Addr);
+    uint32_t ImportLookupTableAddr;
+    error(I.getImportLookupTableRVA(ImportLookupTableAddr));
+    W.printHex("ImportLookupTableRVA", ImportLookupTableAddr);
+    uint32_t ImportAddressTableAddr;
+    error(I.getImportAddressTableRVA(ImportAddressTableAddr));
+    W.printHex("ImportAddressTableRVA", ImportAddressTableAddr);
+    // The import lookup table can be missing with certain older linkers, so
+    // fall back to the import address table in that case.
+    if (ImportLookupTableAddr)
+      printImportedSymbols(I.lookup_table_symbols());
+    else
       printImportedSymbols(I.imported_symbols());
   }
 
   // Delay imports
   for (const DelayImportDirectoryEntryRef &I : Obj->delay_import_directories()) {
     DictScope Import(W, "DelayImport");
     StringRef Name;
     error(I.getName(Name));
     W.printString("Name", Name);