[PATCH] Recognize the armv7 pe magic

Martin Storsjö martin@martin.st
Wed Dec 12 15:20:00 GMT 2012
On Fri, 7 Dec 2012, Martin Storsjö wrote:

> On Tue, 4 Dec 2012, Richard Earnshaw wrote:
>
>> On 29/11/12 14:47, Martin Storsjo wrote:
>>> ---
>>>   include/coff/arm.h |    3 ++-
>>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>> 
>>> ---
>>> Updated the patch to rename it to ARMV7PEMAGIC, which is more consistent
>>> with the existing defines.
>>> 
>>> 
>> 
>> This is OK, though I will note here that I think it is generally a bad idea 
>> to try and encode the architecture variant in the magic number -- it's just 
>> too simplistic.
>
> Can someone commit this?

Ping, can someone please commit this?

// Martin


More information about the Binutils mailing list