Re: nvptx offloading patches [3/n], i386 bits RFD

[Uros Bizjak <ubizjak@gmail.com>]

On Tue, Nov 4, 2014 at 1:35 PM, Bernd Schmidt <bernds@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.lu@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.