[PATCH, Android] Runtime stack protector enabling for Android target
Uros Bizjak
ubizjak@gmail.com
Tue Jul 10 05:56:00 GMT 2012
On Tue, Jul 10, 2012 at 12:40 AM, Maxim Kuvyrkov <maxim@codesourcery.com> wrote:
> On 10/07/2012, at 8:51 AM, Igor Zamyatin wrote:
>
>> Let's try with this patch then. Is it ok for trunk after all
>> appropriate testing is done?
>
> Sorry for the late reply, I've been down with a virus last week.
>
> Uros,
>
> Would you please approve this patch for i386 provided my review below?
>
>>
>> ChangeLog:
>>
>> 2012-07-09 Sergey Melnikov <sergey.melnikov@intel.com>
>>
>> * config/i386/i386.md (stack_protect_set): Disable the pattern
>> for Android since Android libc (bionic) does not provide random
>> value for stack protection guard at gs:0x14. Guard value
>> will be provided from external symbol (default implementation).
>> (stack_protect_set_<mode>): Likewise.
>> (stack_protect_test): Likewise.
>> (stack_protect_test_<mode>): Likewise.
This is OK.
BTW: Since the patch touches only Android specific functionality (and
doesn't affect generic functionality), your approval should be enough.
Thanks,
Uros.
More information about the Gcc-patches
mailing list