[llvm] [llvm-readobj][Object][COFF] Include COFF import file maching type in format string. (PR #78366)

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 00:08:27 PST 2024


https://github.com/mstorsjo approved this pull request.

The change LGTM, thanks (I've also noticed that this is missing, and/or been led to believe that these object files really are architectureless). When inspecting most import libraries, the header/trailer object files still are regular object files, where one is able to see the intended architecture, but this definitely helps and makes things clearer and more testable.

There's a typo in the subject though, `s/maching/machine/`.

https://github.com/llvm/llvm-project/pull/78366


More information about the llvm-commits mailing list