Patch is fine beside one nit.
Line '+#elif defined(_ARM_)'
should be '+#elif defined(_ARM_) || defined (__arm__)'

With that patch is ok.

Thanks,
Kai

2014-05-09 20:28 GMT+02:00 André Hentschel <[email protected]>:
> What about this?
>
> Am 09.05.2014 00:08, schrieb Kai Tietz:
>> Hi
>>
>> I would prefer here to use __x86_64__ __i386__ etc.
>> Have you tested that your changes work for IA targets?
>>
>> Cheers
>> Kai
>>
>> Am 08.05.2014 23:35 schrieb "André Hentschel" <[email protected] 
>> <mailto:[email protected]>>:
>>
>>     Please review, i'll split it up and commit it.
>>
>>     the tab before .globl is needed, otherwise it's not recognized on ARM.
>>     DECLSPEC_ALIGN(8) for the context seems to be hardly supported by our 
>> current toolchain
>>
>
> ------------------------------------------------------------------------------
> Is your legacy SCM system holding you back? Join Perforce May 7 to find out:
> &#149; 3 signs your SCM is hindering your productivity
> &#149; Requirements for releasing software faster
> &#149; Expert tips and advice for migrating your SCM now
> http://p.sf.net/sfu/perforce
> _______________________________________________
> Mingw-w64-public mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
>

------------------------------------------------------------------------------
Is your legacy SCM system holding you back? Join Perforce May 7 to find out:
&#149; 3 signs your SCM is hindering your productivity
&#149; Requirements for releasing software faster
&#149; Expert tips and advice for migrating your SCM now
http://p.sf.net/sfu/perforce
_______________________________________________
Mingw-w64-public mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to