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 <[email protected]>
+
+ * 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 <[email protected]>
* 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