This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Add -mlong-double-128 and make it default for 64-bit Bionic
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: Andrew Pinski <pinskia at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 31 Jan 2014 09:04:56 -0800
- Subject: Re: PATCH: Add -mlong-double-128 and make it default for 64-bit Bionic
- Authentication-results: sourceware.org; auth=none
- References: <20140131135929 dot GA2678 at intel dot com> <CA+=Sn1nJ3fpLQYKi9uiPG2AP7wAfyVAuKoCkCNxhbKyi4Gk9XQ at mail dot gmail dot com> <CAFULd4YURCP4afnta1etTtmRy59WbqpkGiJZkV8mVXUjyD6X8Q at mail dot gmail dot com>
On Fri, Jan 31, 2014 at 9:04 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Fri, Jan 31, 2014 at 5:58 PM, Andrew Pinski <pinskia@gmail.com> wrote:
>> On Fri, Jan 31, 2014 at 5:59 AM, H.J. Lu <hongjiu.lu@intel.com> wrote:
>>> Hi,
>>>
>>> For 64-bit Android, long double is 128-bit IEEE-754 floating point type.
>>> This patch adds -mlong-double-128 to i386 and makes it default for 64-bit
>>> Bionic. I only added MASK_LONG_DOUBLE_128. I made -mlong-double-128,
>>> -mlong-double-64 and -mlong-double-80 negate each other so that the
>>> last one on command line wins. It os OK since we don't support
>>> -mlong-double-xxx in target attribute. I added some testcases to verify
>>> it works correctly. OK for trunk?
>>
>> I don't see any where in the documentation that explains the
>> difference between -mlong-double-128 and -m128bit-long-double. Can
>> you please add that -mlong-double-128 causes soft-floating point.
>> Also what about the interactions between these two options.
>
> The -m128bit-long-double has no effect on TFmode. The names are
> similar by coincidence, but it is a good idea to mention the
> difference in the documentation.
>
I will come up with something after -mlong-double-128 is checked in.
Thanks.
--
H.J.