We had no separate tests for these fields.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
I wouldn't go as far as to say the fields were never tested - they are partially, just not exhaustively tested in ELF/file-headers.test.
llvm/test/tools/llvm-readobj/ELF/file-header-os-abi-version.test | ||
---|---|---|
14 ↗ | (On Diff #234889) | a maximum -> the maximum |
52 ↗ | (On Diff #234889) | I think OS/ABI testing should be split into a separate file from ABIVersion, as the two are separate fields. |
60 ↗ | (On Diff #234889) | Nit: here and throughout, it would be nice if the "OS/ABI" bit lined up: # OSABI-NONE-LLVM: OS/ABI: SystemV (0x0){{$}} # OSABI-NONE-GNU: OS/ABI: UNIX - System V{{$}} |
how the ABI version field