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

[Saleem Abdulrasool via Phabricator via llvm-commits]

compnerd accepted this revision.
compnerd added inline comments.
This revision is now accepted and ready to land.


================
Comment at: tools/llvm-readobj/COFFDumper.cpp:1442
     W.printString("Name", Name);
-    uint32_t Addr;
-    error(I.getImportLookupTableRVA(Addr));
-    W.printHex("ImportLookupTableRVA", Addr);
-    error(I.getImportAddressTableRVA(Addr));
-    W.printHex("ImportAddressTableRVA", Addr);
-    printImportedSymbols(I.imported_symbols());
+    uint32_t ImportLookupTableAddr;
+    error(I.getImportLookupTableRVA(ImportLookupTableAddr));
----------------
ILT would be better I think than ImportLookupTable


================
Comment at: tools/llvm-readobj/COFFDumper.cpp:1445
+    W.printHex("ImportLookupTableRVA", ImportLookupTableAddr);
+    uint32_t ImportAddressTableAddr;
+    error(I.getImportAddressTableRVA(ImportAddressTableAddr));
----------------
IAT would be better than ImportAddressTable I think


https://reviews.llvm.org/D31362