On 6/3/19 11:01 AM, Yoshinori Sato wrote:
> On Sun, 02 Jun 2019 22:12:37 +0900,
> Oleg Endo wrote:
>>
>> On Sun, 2019-06-02 at 20:26 +0900, Yoshinori Sato wrote:
>>> On Fri, 31 May 2019 09:16:18 +0900,
>>> Jeff Law wrote:
>>>>
>>>> On 5/29/19 12:27 PM, Jeff Law wrote:
>>>>> On 5/23/19 6:05 AM, Yoshinori Sato wrote:
>>>>>> I ported linux kernel to Renesas RX.
>>>>>>
>>>>>> rx-*-elf target output a binary different from the standard
>>>>>> ELF.
>>>>>> It has the same format as the Renesas compiler.
>>>>>>
>>>>>> But the linux kernel requires the standard ELF format.
>>>>>> I want to define a rx-*-linux target so that I can generate
>>>>>> a standard ELF binary.
>>>>>
>>>>> Presumably you're resubmitting after your assignment got recorded
>>>>> (I
>>>>> think I saw that fly by recently).
>>>>>
>>>>> I'll construct a ChangeLog and install this on the trunk.
>>>>
>>>> So this is causing libgcc to fail to build for rx-elf.  The problem
>>>> is
>>>> the DF=SF #define.  I think you need so split that out so that it's
>>>> only
>>>> used for rx-linux.
>>>>
>>>> Jeff
>>>
>>> OK. fix it.
>>> I tried build rx-elf target. it success.
>>>
>>
>> Setting DF=SF is the wrong thing to do IMHO.  RX can do DF just fine in
>> software.  If this is hardcoded like that in the roots of the
>> toolchain, it will make compiling packages that actually need real DF
>> completely impossible, won't it?  We also don't set DI = SI just
>> because the hardware is bad at SI ... 
>>
>> Just my 2 cents.
>>
>> Cheers,
>> Oleg
>>
> 
> OK.
> I was misunderstood.
> I think this is not a problem.
[ ... ]
THanks.  Give we already had your original patch in the source tree, I
extracted just the new bits and committed them.  I'm attaching the
changes for archival purposes.

Jeff

commit aafb499477a6f23d01c4c048c20c8833a801396a
Author: law <law@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Wed Jun 5 23:20:27 2019 +0000

            * config.host (rx-*-linux*): Add t-fdpbit to tmake_file
            Add appropriate tm_file clause as well.
            * config/rx/t-rx (HOST_LIBGCC2_CFLAGS): Remove.
    
    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@271978 
138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/libgcc/ChangeLog b/libgcc/ChangeLog
index 82c124f8ece..93cdcda3284 100644
--- a/libgcc/ChangeLog
+++ b/libgcc/ChangeLog
@@ -1,3 +1,9 @@
+2019-06-05  Yoshinori Sato  <ys...@users.sourceforge.jp>
+
+       * config.host (rx-*-linux*): Add t-fdpbit to tmake_file
+       Add appropriate tm_file clause as well.
+       * config/rx/t-rx (HOST_LIBGCC2_CFLAGS): Remove.
+
 2019-06-05  James Clarke  <jrt...@jrtc27.com>
 
        * config/ia64/crtbegin.S (__dso_handle): Put in .sdata/.sbss
diff --git a/libgcc/config.host b/libgcc/config.host
index ff2f0fa1e58..d75e2b5d7aa 100644
--- a/libgcc/config.host
+++ b/libgcc/config.host
@@ -1212,7 +1212,8 @@ rx-*-elf)
        tm_file="$tm_file rx/rx-abi.h rx/rx-lib.h"
        ;;
 rx-*-linux*)
-       tmake_file="rx/t-rx"
+       tmake_file="rx/t-rx t-fdpbit"
+       tm_file="$tm_file rx/rx-lib.h"
        ;;
 s390-*-linux*)
        tmake_file="${tmake_file} s390/t-crtstuff s390/t-linux 
s390/32/t-floattodi t-stack s390/t-stack-s390"
diff --git a/libgcc/config/rx/t-rx b/libgcc/config/rx/t-rx
index 34cdc97cc03..ace3b16f39e 100644
--- a/libgcc/config/rx/t-rx
+++ b/libgcc/config/rx/t-rx
@@ -23,7 +23,6 @@
 # the normal libgcc sources:
 
 LIB2ADD = $(srcdir)/config/rx/rx-abi-functions.c
-HOST_LIBGCC2_CFLAGS += -DDF=SF
 
 
 # We need special handling of the floating point conversion

Reply via email to