This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: PR target/36720: ia64_split_tmode_move doesn't work on little endian
- From: Jim Wilson <wilson at tuliptree dot org>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Steve Ellcey <sje at cup dot hp dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 06 Jul 2008 17:28:37 -0700
- Subject: Re: PATCH: PR target/36720: ia64_split_tmode_move doesn't work on little endian
- References: <6dc9ffc80807031146j1e7fc4acx89ed214401281259@mail.gmail.com>
On Thu, 2008-07-03 at 11:46 -0700, H.J. Lu wrote:
> PR target/36720
> * config/ia64/ia64.c (ia64_split_tmode): Fix typo in TImode
> constant for little endian.
This is OK.
As you mentioned, real_to_target returns values in target word order, so
we should not be trying to swap double words here. The only thing we
need to do is make sure that the 32-bit to 64-bit conversion is correct
with respect to byte order.
I assume you have tested this somehow, perhaps in conjunction with your
__float128 soft-float patch that I haven't looked at yet.
Jim