Re: [PATCH, Android] Stack protector enabling for Android target

2012-05-16 Thread Pavel Chupin
Hi all, First hunk is not needed indeed. Testing is ok. Thanks!!! Here is the final patch: diff --git a/gcc/configure.ac b/gcc/configure.ac index 2c17736..43e760b 100644 --- a/gcc/configure.ac +++ b/gcc/configure.ac @@ -4566,6 +4566,11 @@ AC_CACHE_CHECK(__stack_chk_fail in target C library,

Re: [PATCH, Android] Stack protector enabling for Android target

2012-05-15 Thread Maxim Kuvyrkov
On 12/05/2012, at 9:03 AM, Igor Zamyatin wrote: > Hi! > > Please look at the modified patch in the attachment. ChangeLog remains the > same. > > Tested in android environment(x86_64-*-linux-android), also > bootstrapped on x86_64-unknown-linux-gnu. > I also started regtesting on linux. Is it ok

Re: [PATCH, Android] Stack protector enabling for Android target

2012-05-11 Thread Igor Zamyatin
Hi! Please look at the modified patch in the attachment. ChangeLog remains the same. Tested in android environment(x86_64-*-linux-android), also bootstrapped on x86_64-unknown-linux-gnu. I also started regtesting on linux. Is it ok after successfull regtesting? Thanks, Igor On Mon, May 7, 2012

Re: [PATCH, Android] Stack protector enabling for Android target

2012-05-07 Thread H.J. Lu
On Mon, May 7, 2012 at 11:04 AM, Maxim Kuvyrkov wrote: > On 6/05/2012, at 7:02 PM, Igor Zamyatin wrote: > >> Hi! >> >> The patch enables stack protector for Android. >> Android targets don't contain necessary information in features.h so >> we explicitly enable stack protector for Android. >> >> B

Re: [PATCH, Android] Stack protector enabling for Android target

2012-05-07 Thread Maxim Kuvyrkov
On 6/05/2012, at 7:02 PM, Igor Zamyatin wrote: > Hi! > > The patch enables stack protector for Android. > Android targets don't contain necessary information in features.h so > we explicitly enable stack protector for Android. > > Bootstrapped and regtested on x86_64. Ok to commit? > > Thanks,