[PATCH] D39540: [llvm-nm] Print 'I' for import table data in COFF

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added a comment.

> GNU binutils nm also flags all of the .idata$X chunks as 'i' (while this patch only makes it set on .idata$2 and .idata$6) and also flags impfunction as 'I'.

This is just a consequence of how LLVM treats short import libraries, right? Considering that your patch returns 'i' for any `.idata` section, but short import libraries contain the ILT (`.idata$4`) and IAT (`.idata$5`) sections as `COFFImportFile` rather than `COFFObjectFile`.


https://reviews.llvm.org/D39540