YunQiang Su <yunqiang...@cipunited.com> 于2023年5月12日周五 18:30写道: > > speculation_barrier for MIPS needs sync+jr.hb (r2+), > so we implement __speculation_barrier in libgcc, like arm32 does. > > gcc/ChangeLog: > * config/mips/mips-protos.h (mips_emit_speculation_barrier): New > prototype. > * config/mips/mips.cc (speculation_barrier_libfunc): New static > variable. > (mips_init_libfuncs): Initialize it. > (mips_emit_speculation_barrier): New function. > * config/mips/mips.md (speculation_barrier): Call > mips_emit_speculation_barrier. > * configure.ac: error if gas doesn't accept ssnop for mips1. > * configure: regenerated. > * doc/install.texi: documents mips requires binutils 2.21+. > > libgcc/ChangeLog: > * config/mips/lib1funcs.S: New file. > define __speculation_barrier and include mips16.S. > * config/mips/t-mips: define LIB1ASMSRC as mips/lib1funcs.S. > define LIB1ASMFUNCS as _speculation_barrier. > set version info for __speculation_barrier. > * config/mips/libgcc-mips.ver: New file. > * config/mips/t-mips16: don't define LIB1ASMSRC as mips16.S is > included in lib1funcs.S now.
If no objection, I will commit this V4 patch. > --- > gcc/config/mips/mips-protos.h | 2 + > gcc/config/mips/mips.cc | 13 +++++++ > gcc/config/mips/mips.md | 12 ++++++ > gcc/configure | 32 +++++++++++++++ > gcc/configure.ac | 7 ++++ > gcc/doc/install.texi | 2 + > libgcc/config/mips/lib1funcs.S | 62 ++++++++++++++++++++++++++++++ > libgcc/config/mips/libgcc-mips.ver | 21 ++++++++++ > libgcc/config/mips/t-mips | 7 ++++ > libgcc/config/mips/t-mips16 | 3 +- > 10 files changed, 159 insertions(+), 2 deletions(-) > create mode 100644 libgcc/config/mips/lib1funcs.S > create mode 100644 libgcc/config/mips/libgcc-mips.ver > > diff --git a/gcc/config/mips/mips-protos.h b/gcc/config/mips/mips-protos.h > index 20483469105..da7902c235b 100644 > --- a/gcc/config/mips/mips-protos.h > +++ b/gcc/config/mips/mips-protos.h > @@ -388,4 +388,6 @@ extern void mips_register_frame_header_opt (void); > extern void mips_expand_vec_cond_expr (machine_mode, machine_mode, rtx *); > extern void mips_expand_vec_cmp_expr (rtx *); > > +extern void mips_emit_speculation_barrier_function (void); > + > #endif /* ! GCC_MIPS_PROTOS_H */ > diff --git a/gcc/config/mips/mips.cc b/gcc/config/mips/mips.cc > index ca822758b41..139707fda34 100644 > --- a/gcc/config/mips/mips.cc > +++ b/gcc/config/mips/mips.cc > @@ -13611,6 +13611,9 @@ mips_autovectorize_vector_modes (vector_modes *modes, > bool) > return 0; > } > > + > +static GTY(()) rtx speculation_barrier_libfunc; > + > /* Implement TARGET_INIT_LIBFUNCS. */ > > static void > @@ -13680,6 +13683,7 @@ mips_init_libfuncs (void) > synchronize_libfunc = init_one_libfunc ("__sync_synchronize"); > init_sync_libfuncs (UNITS_PER_WORD); > } > + speculation_barrier_libfunc = init_one_libfunc ("__speculation_barrier"); > } > > /* Build up a multi-insn sequence that loads label TARGET into $AT. */ > @@ -19092,6 +19096,15 @@ mips_avoid_hazard (rtx_insn *after, rtx_insn *insn, > int *hilo_delay, > } > } > > +/* Emit a speculation barrier. > + JR.HB is needed, so we need to put > + speculation_barrier_libfunc in libgcc */ > +void > +mips_emit_speculation_barrier_function () > +{ > + emit_library_call (speculation_barrier_libfunc, LCT_NORMAL, VOIDmode); > +} > + > /* A SEQUENCE is breakable iff the branch inside it has a compact form > and the target has compact branches. */ > > diff --git a/gcc/config/mips/mips.md b/gcc/config/mips/mips.md > index ac1d77afc7d..5d04ac566dd 100644 > --- a/gcc/config/mips/mips.md > +++ b/gcc/config/mips/mips.md > @@ -160,6 +160,8 @@ > ;; The `.insn' pseudo-op. > UNSPEC_INSN_PSEUDO > UNSPEC_JRHB > + > + VUNSPEC_SPECULATION_BARRIER > ]) > > (define_constants > @@ -7455,6 +7457,16 @@ > mips_expand_conditional_move (operands); > DONE; > }) > + > +(define_expand "speculation_barrier" > + [(unspec_volatile [(const_int 0)] VUNSPEC_SPECULATION_BARRIER)] > + "" > + " > + mips_emit_speculation_barrier_function (); > + DONE; > + " > +) > + > > ;; > ;; .................... > diff --git a/gcc/configure b/gcc/configure > index 191f68581b3..e55b40d7f4c 100755 > --- a/gcc/configure > +++ b/gcc/configure > @@ -28699,6 +28699,38 @@ $as_echo "#define HAVE_LD_PERSONALITY_RELAXATION 1" > >>confdefs.h > > fi > > + { $as_echo "$as_me:${as_lineno-$LINENO}: checking assembler for mips1 > recognize ssnop" >&5 > +$as_echo_n "checking assembler for mips1 recognize ssnop... " >&6; } > +if ${gcc_cv_as_mips1_ssnop+:} false; then : > + $as_echo_n "(cached) " >&6 > +else > + gcc_cv_as_mips1_ssnop=no > + if test x$gcc_cv_as != x; then > + $as_echo 'ssnop' > conftest.s > + if { ac_try='$gcc_cv_as $gcc_cv_as_flags -mabi=32 -mips1 -o conftest.o > conftest.s >&5' > + { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5 > + echo $ac_try > + (eval $ac_try) 2>&5 > + ac_status=$? > + $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5 > + test $ac_status = 0; }; } > + then > + gcc_cv_as_mips1_ssnop=yes > + else > + echo "configure: failed program was" >&5 > + cat conftest.s >&5 > + fi > + rm -f conftest.o conftest.s > + fi > +fi > +{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $gcc_cv_as_mips1_ssnop" >&5 > +$as_echo "$gcc_cv_as_mips1_ssnop" >&6; } > + > + > + if test x$gcc_cv_as_mips1_ssnop = xno; then > + as_fn_error $? "New binutils is required to support ssnop even for > MIPS I: 2.21+" "$LINENO" 5 > + fi > + > { $as_echo "$as_me:${as_lineno-$LINENO}: checking assembler for -mnan= > support" >&5 > $as_echo_n "checking assembler for -mnan= support... " >&6; } > if ${gcc_cv_as_mips_nan+:} false; then : > diff --git a/gcc/configure.ac b/gcc/configure.ac > index 075424669c9..6d2cb4c51a0 100644 > --- a/gcc/configure.ac > +++ b/gcc/configure.ac > @@ -5270,6 +5270,13 @@ EOF > pointers into PC-relative form.]) > fi > > + gcc_GAS_CHECK_FEATURE([mips1 recognize ssnop], gcc_cv_as_mips1_ssnop, > + [-mabi=32 -mips1], [ssnop],,,) > + if test x$gcc_cv_as_mips1_ssnop = xno; then > + AC_MSG_ERROR( > + [New binutils is required to support ssnop even for MIPS I: 2.21+]) > + fi > + > gcc_GAS_CHECK_FEATURE([-mnan= support], > gcc_cv_as_mips_nan, > [-mnan=2008],,, > diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi > index fa91ce1953d..fae6ec5f7a9 100644 > --- a/gcc/doc/install.texi > +++ b/gcc/doc/install.texi > @@ -4610,6 +4610,8 @@ the use of break, use the @option{--with-divide=breaks} > @command{configure} option when configuring GCC@. The default is to > use traps on systems that support them. > > +GNU Binutils 2.21 or newer is required. > + > @html > <hr /> > @end html > diff --git a/libgcc/config/mips/lib1funcs.S b/libgcc/config/mips/lib1funcs.S > new file mode 100644 > index 00000000000..12314953a7f > --- /dev/null > +++ b/libgcc/config/mips/lib1funcs.S > @@ -0,0 +1,62 @@ > +/* Copyright (C) 2023 Free Software Foundation, Inc. > + > +This file is free software; you can redistribute it and/or modify it > +under the terms of the GNU General Public License as published by the > +Free Software Foundation; either version 3, or (at your option) any > +later version. > + > +This file is distributed in the hope that it will be useful, but > +WITHOUT ANY WARRANTY; without even the implied warranty of > +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > +General Public License for more details. > + > +Under Section 7 of GPL version 3, you are granted additional > +permissions described in the GCC Runtime Library Exception, version > +3.1, as published by the Free Software Foundation. > + > +You should have received a copy of the GNU General Public License and > +a copy of the GCC Runtime Library Exception along with this program; > +see the files COPYING3 and COPYING.RUNTIME respectively. If not, see > +<http://www.gnu.org/licenses/>. */ > + > +//#include "mips16.S" > + > +#ifdef L_speculation_barrier > + > +/* MIPS16e1 has no sync/jr.hb instructions, and MIPS16e2 lacks of jr.hb. > + So, we use normal MIPS code here, just like what we do for __sync_* */ > + .set nomips16 > + > + .set noreorder > + .globl __speculation_barrier > + .ent __speculation_barrier > + > +__speculation_barrier: > + .set push > +#if __mips >= 2 || defined(__linux) > + /* MIPS1 has no sync. while Linux can trap&emu sync */ > + /* FIXME: Will somebody use linux/gcc for MIPS1/baremetal? */ > + sync /* complementation barrier for memory */ > +#endif > + > + > +#if __mips_isa_rev >= 1 > + /* Binutils claims that JR in R1 can do same as jr.hb. > + R6 changes the encoding of jr.hb */ > + jr.hb $ra /* Jump with instruction hazard barrier */ > +#else > + /* ssnop is actually available since R5500, > + and it will be decoded as nop on earlier processors. > + gas can recognize it even with -march=mips1 since 2.21 */ > + /* MIPS1 to MIPSr1: R10000 have 7 stage pipeline, > + so 8 ssnop is sufficient to block all speculation on all CPUs */ > + .rept 8 > + ssnop > + .endr > + .word 0x03e00408 /* The encoding of jr.hb $ra */ > +#endif > + .set pop > + .end __speculation_barrier > + > + .set reorder > +#endif > diff --git a/libgcc/config/mips/libgcc-mips.ver > b/libgcc/config/mips/libgcc-mips.ver > new file mode 100644 > index 00000000000..68f8d2bbd51 > --- /dev/null > +++ b/libgcc/config/mips/libgcc-mips.ver > @@ -0,0 +1,21 @@ > +# Copyright (C) 2023 Free Software Foundation, Inc. > +# > +# This file is part of GCC. > +# > +# GCC is free software; you can redistribute it and/or modify > +# it under the terms of the GNU General Public License as published by > +# the Free Software Foundation; either version 3, or (at your option) > +# any later version. > +# > +# GCC is distributed in the hope that it will be useful, > +# but WITHOUT ANY WARRANTY; without even the implied warranty of > +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > +# GNU General Public License for more details. > +# > +# You should have received a copy of the GNU General Public License > +# along with GCC; see the file COPYING3. If not see > +# <http://www.gnu.org/licenses/>. > + > +GCC_14.0 { > + __speculation_barrier > +} > diff --git a/libgcc/config/mips/t-mips b/libgcc/config/mips/t-mips > index 4fb8e136217..d05ef7cbf74 100644 > --- a/libgcc/config/mips/t-mips > +++ b/libgcc/config/mips/t-mips > @@ -7,3 +7,10 @@ softfp_truncations := > softfp_exclude_libgcc2 := n > > LIB2ADD_ST += $(srcdir)/config/mips/lib2funcs.c > + > + > +LIB1ASMSRC = mips/lib1funcs.S > +LIB1ASMFUNCS = _speculation_barrier > + > +# Version these symbols if building libgcc.so. > +SHLIB_MAPFILES += $(srcdir)/config/mips/libgcc-mips.ver > diff --git a/libgcc/config/mips/t-mips16 b/libgcc/config/mips/t-mips16 > index 2bad5119d51..5fd9d60d7a3 100644 > --- a/libgcc/config/mips/t-mips16 > +++ b/libgcc/config/mips/t-mips16 > @@ -16,8 +16,7 @@ > # along with GCC; see the file COPYING3. If not see > # <http://www.gnu.org/licenses/>. > > -LIB1ASMSRC = mips/mips16.S > -LIB1ASMFUNCS = _m16addsf3 _m16subsf3 _m16mulsf3 _m16divsf3 \ > +LIB1ASMFUNCS += _m16addsf3 _m16subsf3 _m16mulsf3 _m16divsf3 \ > _m16eqsf2 _m16nesf2 _m16gtsf2 _m16gesf2 _m16lesf2 _m16ltsf2 \ > _m16unordsf2 \ > _m16fltsisf _m16fix_truncsfsi _m16fltunsisf \ > -- > 2.30.2 >