On Fri, Feb 24, 2012 at 11:22 AM, Ilya Enkovich <enkovich....@gmail.com> wrote:
>> On Wed, Feb 22, 2012 at 3:57 PM, Ilya Enkovich <enkovich....@gmail.com> 
>> wrote:
>>> Hello,
>>>
>>> Here is a simple patch which enables exceptions and RTTI by default
>>> for Android target. OK for trunk?
>>
>> Err - isn't that the default?  Thus, simply delete the bogus spec?
>>
>> Richard.
>>
>>
> Hi,
>
> Is following patch OK or it's better to remove whole macro and its usages?

The latter.

Richard.

> Thanks,
> Ilya
> --
> 2012-02-22  Enkovich Ilya  <ilya.enkov...@intel.com>
>
>        * gcc/config/linux-android.h (ANDROID_CC1PLUS_SPEC): Enable
>        exceptions and rtti by default.
>
>
> diff --git a/gcc/config/linux-android.h b/gcc/config/linux-android.h
> index 94c5274..180b62b 100644
> --- a/gcc/config/linux-android.h
> +++ b/gcc/config/linux-android.h
> @@ -45,9 +45,7 @@
>   "%{!mglibc:%{!muclibc:%{!mbionic: -mbionic}}} "                      \
>   "%{!fno-pic:%{!fno-PIC:%{!fpic:%{!fPIC: -fPIC}}}}"
>
> -#define ANDROID_CC1PLUS_SPEC                                           \
> -  "%{!fexceptions:%{!fno-exceptions: -fno-exceptions}} "               \
> -  "%{!frtti:%{!fno-rtti: -fno-rtti}}"
> +#define ANDROID_CC1PLUS_SPEC ""
>
>  #define ANDROID_LIB_SPEC \
>   "%{!static: -ldl}"

Reply via email to