On 15/11/2019 19:22, Segher Boessenkool wrote:
> On Fri, Nov 15, 2019 at 06:58:24PM +0000, Szabolcs Nagy wrote:
>> 2019-11-15  Szabolcs Nagy  <szabolcs.n...@arm.com>
>>
>>      * configure.ac (gcc_cv_target_ldbl128): Set for *-musl* targets.
> 
> That is not what the patch does.  It sets it to yes for s390*-linux-musl*,
> it sets it to no for powerpc*-*-linux-musl*, and it doesn't do anything for
> other *-musl* configurations.

ok i can fix the changelog to:

        * configure.ac (gcc_cv_target_ldbl128): Set for powerpc*-*-linux-musl*
        and s390*-*-linux-musl* targets.

(but i think there is no other *-musl* target that uses gcc_cv_target_ldbl128
so the original was not entirely wrong either)

> 
> The powerpc part is okay for trunk, if this is what musl wants to do.
> (musl has no OS port maintainer listed in MAINTAINERS, maybe that should
> be fixed?)
> 
> 
> Segher
> 

Reply via email to