[Pixman] [PATCH 1/2] configure.ac: Use '-mloongson-mmi' for Loongson MMI.

[Shiyou Yin]

>-----Original Message-----
>From: YunQiang Su [mailto:wzssyqa at gmail.com]
>Sent: Saturday, March 28, 2020 11:17 PM
>To: Yin Shiyou
>Cc: pixman
>Subject: Re: Re: [Pixman] [PATCH 1/2] configure.ac: Use '-mloongson-mmi' for Loongson MMI.
>
><yinshiyou-hf at loongson.cn> 于2020年3月28日周六 上午11:13写道：
>>
>> > -----原始邮件-----
>> > 发件人: "YunQiang Su" <wzssyqa at gmail.com>
>> > 发送时间: 2020-03-28 08:50:46 (星期六)
>> > 收件人: "Shiyou Yin" <yinshiyou-hf at loongson.cn>
>> > 抄送: pixman <pixman at lists.freedesktop.org>
>> > 主题: Re: [Pixman] [PATCH 1/2] configure.ac: Use '-mloongson-mmi' for Loongson MMI.
>> >
>> > Please reject ALL of @loongson.cn patch, as I know they are try to
>> > KILL all other MIPS platform.
>> > There are some extensions are Loongson-only (MMI here), while they
>> > continue to try to set them as default.
>>
>> Please don't be so extreme, we will never trying to kill other mips platform.
>
>This a the mistake of mine.
>The LS_CFLAGS is used only for libpixman-loongson-mmi.la,
>and it will not be used globally. Then it won't break other MIPS platforms.
>I apologize for my mistake.
>
>> About this setting for MMI, yes, now it's enable by default, but there are runtime check for MMI.
>> If it still crashes on other mips platform, it may be a bug of runtimechek.
>
>For the incompatible between 2F (-march=loongson2f) and 3A (-mloongson-mmi),
>I checked that all of currently used insns are compatible.
>
>While there do be some insns are incompatible: single-paired and some
>condition insns.
>Luckily, they are not used yet.
>If they are used in future, we should be very carefully.
>
>
>> I still suggest to enable it by default. If our views cannot be reached, then choose proposal B
>> to delete this default setting.
>
>--
>YunQiang Su

Hi Matt,
A consensus was finally reached on this patch.
Please help to merge proposal A.
For your convenience, I attached it in attachment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: proposal_A_0001-configure.ac-use-mloongson-mmi-for-Loongson-MMI.patch
Type: application/octet-stream
Size: 1209 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/pixman/attachments/20200401/60d8ac52/attachment.obj>