Hi!

On Mon, Apr 06, 2020 at 10:35:34PM +0200, Sebastian Huber wrote:
> What do you think about the attached patch?

(Please use a correct MIME type for attachments (x-* never is correct on
mailing lists.  Just text/plain will do fine.)

> libgcc/
> 
>       * config/rs6000/crtresfpr.S: Disable all functions if
>       _SOFT_DOUBLE is defined.

Can't you just use an appropriate .machine?  I'd prefer that, that's how
AltiVec stuff is done, for example.

>         * config/rs6000/crtresxfpr.S: Likewise.
>         * config/rs6000/crtsavfpr.S: Likewise.

(Broken indentation).

> +/* On PowerPC64 Linux, these functions are provided by the linker.  
> Soft-double
> + * configurations do not need these functions. */
> +#if !defined(__powerpc64__) && !defined(_SOFT_DOUBLE)

We use __NO_FPRS__ more often.  All of these are the same now, but this
name makes more sense as well ;-)


Segher

Reply via email to