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 | a maximum -> the maximum | |
52 | I think OS/ABI testing should be split into a separate file from ABIVersion, as the two are separate fields. | |
60 | 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{{$}} |
llvm/test/tools/llvm-readobj/ELF/file-header-abi-version.test | ||
---|---|---|
39 ↗ | (On Diff #235123) | The comment may be redundant... As 52 and 0x34 appear on adjacent lines above. It should be obvious. |
llvm/test/tools/llvm-readobj/ELF/file-header-os-abi.test | ||
8 ↗ | (On Diff #235123) | It looks like {{$}} can be avoided by using FileCheck --match-full-line. |
a maximum -> the maximum