Thanks, that looks better IMHO. > +;; Copyright (C) 2011-2024 Free Software Foundation, Inc. > +;; Contributed by Andrew Waterman (and...@sifive.com). > +;; Based on MIPS target for GNU compiler.
You might want to change that, as well as the date. While at it you can also fix the broken date in my original file ;) > +(define_insn_reservation "vec_load" 6 > + (and (eq_attr "is_inorder" "no") > + (eq_attr "type" "vlde,vldm,vlds,vldux,vldox,vldff,vldr")) > + "vxu_ooo_issue,vxu_ooo_alu") I would rather ditch the is_inorder attribute for now and define "low" latencies as well as reservations explicitly once we're sure rather than falling back to scheduler defaults. OK with those changes. Regards Robin
