[cfe-commits] [PATCH] Enable UseInitArray for Android X86/MIPS

[Logan Chien]

Dear all,

  This is the revised patch which renamed "androideabi" to "android"
to avoid confusion.  Please have a look.  Thanks.

Sincerely,
Logan

p.s. Here's the link to the patch:
https://github.com/loganchien/clang-logan/commit/b16a2b09fc54f21e326c81c7a7e4f61360722dbf

On Thu, Aug 30, 2012 at 3:15 PM, Logan Chien <tzuhsiang.chien at gmail.com>wrote:

> Hi Tim,
>
>     Thanks for your reply.  androideabi is not for MIPS or X86.
> I have an upcoming patch to rename all of the usage.  In fact
> some friend from MIPS suggested me not to use the "EABI"
> to avoid confusion.  Thanks.
>
> Logan
>
>
> On Thu, Aug 30, 2012 at 3:05 PM, Tim Northover <t.p.northover at gmail.com>wrote:
>
>> On Thu, Aug 30, 2012 at 7:38 AM, Logan Chien <tzuhsiang.chien at gmail.com>
>> wrote:
>> > Sorry, I have attached the wrong patch.  This is the correct one.
>>  Please
>> > have a look.  Thanks.
>>
>> I'm not involved in android development, but is "androideabi" really
>> used on x86 and Mips? It looks like the name involves "EABI" which is
>> a specific ARM ABI. Your test seems to be the first to apply it to
>> non-ARM targets, which may be a sensible thing to do (everyone and his
>> brother uses the ia-64 ABI these days) but seems a little odd.
>>
>> Anyone else know more about Android convention?
>>
>> Tim.
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120831/070fea1d/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-Enable-fuse-init-array-for-Android-X86-MIPS.patch
Type: application/octet-stream
Size: 2204 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120831/070fea1d/attachment.obj>