On Fri, Jan 31, 2014 at 9:04 AM, Uros Bizjak <ubiz...@gmail.com> wrote: > On Fri, Jan 31, 2014 at 5:58 PM, Andrew Pinski <pins...@gmail.com> wrote: >> On Fri, Jan 31, 2014 at 5:59 AM, H.J. Lu <hongjiu...@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.