From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ciao.gmane.io (ciao.gmane.io [116.202.254.214]) by sourceware.org (Postfix) with ESMTPS id A360F386183E for ; Wed, 20 Dec 2023 22:55:51 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A360F386183E Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=rivosinc.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=m.gmane-mx.org ARC-Filter: OpenARC Filter v1.0.0 sourceware.org A360F386183E Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=116.202.254.214 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1703112952; cv=none; b=tDASU/2KTYzsNsiZDoelaWrIl4OfQVsLeuwk2iDwoQjL4mjLJtAoMC6wZbsN+iEAzgt6d+DmbeQyyQEBeT0PtJVbPgGkDF+LLJp5FkStpDBuVWT74q4A5XZmVU0J+me1Sagdcuk0Bapm0dFVTCI80McAMYFIiF97l8eOy3ySvZg= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1703112952; c=relaxed/simple; bh=PDP6sYZB+uUYtyZLI/2iFB5RIBAdd0LpOvV89MKOa3g=; h=To:From:Subject:Date:Message-ID:Mime-Version; b=RlHMzIXW/htprL8hLU+rWBbMbdfLhGkgzfHa36jZRB5gIKenqmoPWGXNkPVcXpcrSBPSfKstVzHGVBUCUWbknXgdoPJzvH1v644NNQuoK14/vye/hwNk8PF2SGoupC0ZFmoPBV7qIG+LJHDxJ7oEqQIE9HGzdAu8QH/+RaWABmg= ARC-Authentication-Results: i=1; server2.sourceware.org Received: from list by ciao.gmane.io with local (Exim 4.92) (envelope-from ) id 1rG5Ti-0003Jd-B8 for gcc-patches@gcc.gnu.org; Wed, 20 Dec 2023 23:55:50 +0100 X-Injected-Via-Gmane: http://gmane.org/ To: gcc-patches@gcc.gnu.org From: Edwin Lu Subject: Re: [PATCH 2/3][RFC] RISC-V: Add vector related reservations Date: Wed, 20 Dec 2023 14:55:41 -0800 Message-ID: <5f1b3ce3-e699-4fbc-b680-b56085af0df4@rivosinc.com> References: <20231215185328.794425-1-ewlu@rivosinc.com> <20231215185328.794425-3-ewlu@rivosinc.com> <39a1a8fb-6d6f-4c81-a4df-5d658467e34f@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit User-Agent: Mozilla Thunderbird Cc: gnu-toolchain@rivosinc.com, kito.cheng@gmail.com, Robin Dapp Content-Language: en-US In-Reply-To: <39a1a8fb-6d6f-4c81-a4df-5d658467e34f@gmail.com> X-Spam-Status: No, score=-8.1 required=5.0 tests=BAYES_00,BODY_8BITS,GIT_PATCH_0,HEADER_FROM_DIFFERENT_DOMAINS,KAM_DMARC_STATUS,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Message-ID: <20231220225541.G_mKbfW9uJb8f-AOfJRUheaJHRHSyXO7l0QL8_1Htio@z> On 12/20/2023 10:57 AM, Jeff Law wrote: > > > On 12/15/23 11:53, Edwin Lu wrote: >> This patch copies the vector reservations from generic-ooo.md and >> inserts them into generic.md and sifive.md. The vector pipelines are >> necessary to avoid an ICE from the assert > >> >> gcc/ChangeLog: >> >>     * config/riscv/generic-ooo.md: syntax >>     * config/riscv/generic.md (pipe0): new reservation >>     (generic_vec_load): ditto >>     (generic_vec_store): ditto >>     (generic_vec_loadstore_seg): ditto >>     (generic_generic_vec_alu): ditto >>     (generic_vec_fcmp): ditto >>     (generic_vec_imul): ditto >>     (generic_vec_fadd): ditto >>     (generic_vec_fmul): ditto >>     (generic_crypto): ditto >>     (generic_vec_perm): ditto >>     (generic_vec_reduction): ditto >>     (generic_vec_ordered_reduction): ditto >>     (generic_vec_idiv): ditto >>     (generic_vec_float_divsqrt): ditto >>     (generic_vec_mask): ditto >>     (generic_vec_vesetvl): ditto >>     (generic_vec_setrm): ditto >>     (generic_vec_readlen): ditto >>     * config/riscv/sifive-7.md (sifive_7): new reservation >>     (sifive_7_vec_load): ditto >>     (sifive_7_vec_store): ditto >>     (sifive_7_vec_loadstore_seg): ditto >>     (sifive_7_sifive_7_vec_alu): ditto >>     (sifive_7_vec_fcmp): ditto >>     (sifive_7_vec_imul): ditto >>     (sifive_7_vec_fadd): ditto >>     (sifive_7_vec_fmul): ditto >>     (sifive_7_crypto): ditto >>     (sifive_7_vec_perm): ditto >>     (sifive_7_vec_reduction): ditto >>     (sifive_7_vec_ordered_reduction): ditto >>     (sifive_7_vec_idiv): ditto >>     (sifive_7_vec_float_divsqrt): ditto >>     (sifive_7_vec_mask): ditto >>     (sifive_7_vec_vesetvl): ditto >>     (sifive_7_vec_setrm): ditto >>     (sifive_7_vec_readlen): ditto >> >> Signed-off-by: Edwin Lu >> Co-authored-by: Robin Dapp >> --- >>   gcc/config/riscv/generic-ooo.md |  19 ++--- >>   gcc/config/riscv/generic.md     | 118 ++++++++++++++++++++++++++++++++ >>   gcc/config/riscv/sifive-7.md    | 118 ++++++++++++++++++++++++++++++++ >>   3 files changed, 242 insertions(+), 13 deletions(-) >> >> diff --git a/gcc/config/riscv/generic-ooo.md >> b/gcc/config/riscv/generic-ooo.md >> index de93245f965..18b606bb981 100644 >> --- a/gcc/config/riscv/generic-ooo.md >> +++ b/gcc/config/riscv/generic-ooo.md > I'm not sure why you changed these.  In general we try to keep lines > under 80 columns in the source.  Things like insn reservations are in a > grey area as the lists sometimes get very long.  In general I'd leave > this stuff alone if it doesn't have a function change. > Hmm when I was testing things before, I encountered an error where the scheduler had a problem with "\ " with the assert enabled, but now I can't reproduce it. I'll revert it back to what it originally was and experiment some more. >> >> index 3e49d942495..7ac974ad634 100644 >> --- a/gcc/config/riscv/generic.md >> +++ b/gcc/config/riscv/generic.md > Note that some of the stuff pointed out on patch #1 applies here to, > like rdfrm not being a vector load/store.  So as you clean up patch #1, > make sure to mirror the cleanups in patch #2 of the series. > Will do! Edwin