That looks generally OK, though it lacks ChangeLog entries. But I think it's a bit weird to declare those union types for the sole purpose of using sizeof on them. I think it would be cleaner to replace e.g. sizeof (phdr_u) with:
#define PHDR_MAX_SIZE MAX (sizeof (Elf32_Phdr), sizeof (Elf64_Phdr))
