On Tue, Nov 4, 2014 at 1:35 PM, Bernd Schmidt <ber...@codesourcery.com> wrote:
>> Not sure how to deal with this any further out than the immediate term
>> than using a hack like this. Though I'd prefer to avoid the #ifdef as it
>> seems to me this shouldn't be baked in at build/configure time.
>
>
> Yeah, I'm not expecting the i386 part to go in quite as-is. For reference
> I'm including the offload-abi patch - Ilya is submitting this along with
> other option changes. One possibility would be to print and recognize
> strings such as lp64D128 or lp64D96 which would include information about
> the size of long double. Somehow though I can't really bring myself to
> believe that -mlong-double128 is a real use case with offloading so we might
> just disallow the combination.
>
> CCing Uros in case he has an opinion.
-mlong-double-128 was introduced for Android in:
2014-02-03 H.J. Lu <hongjiu...@intel.com>
* config/i386/i386.c (flag_opts): Add -mlong-double-128.
(ix86_option_override_internal): Default long double to 64-bit for
32-bit Bionic and to 128-bit for 64-bit Bionic.
[...]
IMO, if it troubles offloading, anything else than the default
-mlong-double-80 should be disallowed.
Uros.
