[PATCH] D24471: [ELF] Set EF_ARM_EABI_VER5 for ARM

Martin Storsjö via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 13 04:11:12 PDT 2016


mstorsjo added a comment.

In https://reviews.llvm.org/D24471#540599, @emaste wrote:

> Can you add a test case?


It seems to me that llvm-objdump doesn't print e_flags at all anywhere. Any suggestions on how to test it without that?

Also, is something similar to "Subtarget->isTargetAEABI() || Subtarget->isTargetGNUAEABI() || Subtarget->isTargetMuslAEABI()" available in ELF/Writer.cpp? In that case, we could selectively set the flag in those cases.


https://reviews.llvm.org/D24471





More information about the llvm-commits mailing list