Re: [RFC PATCH] Implementing ifunc target hook

[Alexander Ivchenko]

Yep.. we missed that:

t-linux-android was added here:
  # Add Android userspace support to Linux targets.
  case $target in
    *linux*)
      tm_p_file="${tm_p_file} linux-protos.h"
      tmake_file="${tmake_file} t-linux-android"
      tm_file="$tm_file linux-android.h"
      extra_options="$extra_options linux-android.opt"
      extra_objs="$extra_objs linux-android.o"
      ;;
  esac

and here:

arm*-*-linux-*)                 # ARM GNU/Linux with ELF
        tm_file="dbxelf.h elfos.h gnu-user.h linux.h linux-android.h
glibc-stdint.h arm/elf.h arm/linux-gas.h arm/linux-elf.h"
        case $target in
        arm*b-*-linux*)
            tm_defines="${tm_defines} TARGET_BIG_ENDIAN_DEFAULT=1"
            ;;
        esac
        tmake_file="${tmake_file} arm/t-arm arm/t-arm-elf arm/t-bpabi
arm/t-linux-eabi t-linux-android"
        tm_file="$tm_file arm/bpabi.h arm/linux-eabi.h arm/aout.h arm/arm.h"
        extra_objs="$extra_objs linux-android.o"
        # Define multilib configuration for arm-linux-androideabi.
        case ${target} in
        *-androideabi)
            tmake_file="$tmake_file arm/t-linux-androideabi"
            ;;
        esac



I deleted the second encounter:

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d18c6e9..0e1d5e4 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2013-04-02  Alexander Ivchenko  <alexander.ivche...@intel.com>
+
+       * config.gcc (arm*-*-linux-*): Remove duplicate t-linux-android.
+
 2013-04-02  Richard Biener  <rguent...@suse.de>

        PR tree-optimization/56778
diff --git a/gcc/config.gcc b/gcc/config.gcc
index e51db91..44ed190 100644
--- a/gcc/config.gcc
+++ b/gcc/config.gcc
@@ -878,7 +878,7 @@ arm*-*-linux-*)                     # ARM GNU/Linux with ELF
            tm_defines="${tm_defines} TARGET_BIG_ENDIAN_DEFAULT=1"
            ;;
        esac
-       tmake_file="${tmake_file} arm/t-arm arm/t-arm-elf arm/t-bpabi
arm/t-linux-eabi t-linux-android"
+       tmake_file="${tmake_file} arm/t-arm arm/t-arm-elf arm/t-bpabi
arm/t-linux-eabi"
        tm_file="$tm_file arm/bpabi.h arm/linux-eabi.h arm/aout.h arm/arm.h"
        extra_objs="$extra_objs linux-android.o"
        # Define multilib configuration for arm-linux-androideabi.


is it ok?

thanks,
Alexander

2013/4/2 Jakub Jelinek <ja...@redhat.com>:
> On Wed, Mar 27, 2013 at 01:56:48PM +0400, Kirill Yukhin wrote:
>> >
>> > Otherwise OK.
>> >
>> > Thanks,
>>
>> Hi,  chacked into trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-03/msg00785.html
>
> This leads to:
> ../../gcc/config/t-linux-android:22: warning: overriding recipe for target 
> `linux-android.o'
> ../../gcc/config/t-linux-android:22: warning: ignoring old recipe for target 
> `linux-android.o'
> for arm*-linux* target (cross in my case).  t-linux-android is listed twice.
>
>         Jakub