>-----Original Message----- >From: YunQiang Su [mailto:[email protected]] >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. > ><[email protected]> 于2020年3月28日周六 上午11:13写道： >> >> > -----原始邮件----- >> > 发件人: "YunQiang Su" <[email protected]> >> > 发送时间: 2020-03-28 08:50:46 (星期六) >> > 收件人: "Shiyou Yin" <[email protected]> >> > 抄送: pixman <[email protected]> >> > 主题: 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.
proposal_A_0001-configure.ac-use-mloongson-mmi-for-Loongson-MMI.patch
Description: Binary data
_______________________________________________ Pixman mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/pixman
