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 E60CB385843E for ; Fri, 25 Aug 2023 23:45:09 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org E60CB385843E 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 Received: from list by ciao.gmane.io with local (Exim 4.92) (envelope-from ) id 1qZgUG-0009iY-In for gcc-patches@gcc.gnu.org; Sat, 26 Aug 2023 01:45:08 +0200 X-Injected-Via-Gmane: http://gmane.org/ To: gcc-patches@gcc.gnu.org From: Edwin Lu Subject: Re: [Committed] RISC-V: Add Types to Un-Typed Sync Instructions: Date: Fri, 25 Aug 2023 16:44:57 -0700 Message-ID: References: <20230824211957.671151-1-ewlu@rivosinc.com> <5a02fd13-95af-768a-f61d-7daf890076fa@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 Content-Language: en-US In-Reply-To: <5a02fd13-95af-768a-f61d-7daf890076fa@gmail.com> X-Spam-Status: No, score=-1.0 required=5.0 tests=BAYES_00,BODY_8BITS,HEADER_FROM_DIFFERENT_DOMAINS,KAM_DMARC_STATUS,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=no 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: <20230825234457.ypDjebsiB1X3AzUhHHpxXZak_rxm1Bm1z05ZAaLGMqQ@z> On 8/25/2023 11:53 AM, Jeff Law via Gcc-patches wrote: > > > On 8/24/23 15:19, Edwin Lu wrote: >> Related Discussion: >> https://inbox.sourceware.org/gcc-patches/12fb5088-3f28-0a69-de1e-f387371a5eb2@gmail.com/ >> >> This patch updates the sync instructions to ensure that no insn is left >> without a type attribute. Updates a total of 6 insns to have type >> "atomic" >> >> Tested for regressions using rv32/64 multilib with newlib/linux. >> >> gcc/Changelog: >> >>     * config/riscv/sync-rvwmo.md: updated types to "multi" or >>      "atomic" based on number of assembly lines generated >>     * config/riscv/sync-ztso.md: likewise >>     * config/riscv/sync.md: likewise > OK. > > You should have your write access set up already.  So go ahead and > follow the directions in the email you received to get yourself into the > MAINTAINERS file.  Then you can go ahead and push this change. > > THanks, > Jeff > Committed! Edwin