On Thu, Mar 26, 2020 at 5:57 AM Shiyou Yin <yinshiyou...@loongson.cn> wrote: > > It's recommended to use '-mloongson-mmi' for MMI. > --- > configure.ac | 2 +- > meson.build | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure.ac b/configure.ac > index 1ca3974..fd7df47 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -273,7 +273,7 @@ dnl > =========================================================================== > dnl Check for Loongson Multimedia Instructions > > if test "x$LS_CFLAGS" = "x" ; then > - LS_CFLAGS="-march=loongson2f" > + LS_CFLAGS="-mloongson-mmi" > fi > > have_loongson_mmi=no > diff --git a/meson.build b/meson.build > index 15d3409..a45c969 100644 > --- a/meson.build > +++ b/meson.build > @@ -51,7 +51,7 @@ endforeach > > use_loongson_mmi = get_option('loongson-mmi') > have_loongson_mmi = false > -loongson_mmi_flags = ['-march=loongson2f'] > +loongson_mmi_flags = ['-mloongson-mmi'] > if not use_loongson_mmi.disabled() > if host_machine.cpu_family() == 'mips64' and cc.compiles(''' > #ifndef __mips_loongson_vector_rev > --
Thanks very much. This looks good to me. My only (minor) concern is that the -mloongson-mmi flag is only available since GCC 9, but likely any users would need to change -march=loongson2f to -march=loongson3a anyway, and they can easily change -mloongson-mmi back to -march=... if needed. I'll just double check that with this patch that the test suite passes on my Yeeloong and then commit it. (and sorry for my delayed response) _______________________________________________ Pixman mailing list Pixman@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/pixman