From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl1-x642.google.com (mail-pl1-x642.google.com [IPv6:2607:f8b0:4864:20::642]) by sourceware.org (Postfix) with ESMTPS id 52C4F3AAA06A for ; Tue, 2 May 2023 20:19:10 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 52C4F3AAA06A Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=rivosinc.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=rivosinc.com Received: by mail-pl1-x642.google.com with SMTP id d9443c01a7336-1ab01bf474aso17833285ad.1 for ; Tue, 02 May 2023 13:19:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=rivosinc-com.20221208.gappssmtp.com; s=20221208; t=1683058750; x=1685650750; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=R8R44OCBFHsY7LmipoXmH56AiKRTTg8SVS3Xfg7OSf0=; b=eMo3fxpzwpqwoxcnQCu8ffoJ1syfqvlQ03WxBxPId8zNahK3GDaW88pDR6AeapXN0g Dz5jFauiKWl3/9Dwf5S+4JWKXMvWvilMaXkT7+DCi8Ki7x4ZAGKT7P0PCeVTrs3369/T uCpd+wMc+mCnzO7yZ7bcGPQ+mfM03tgeef9HcSKcnKFYgi/IdmiD4hiEFTxIPwWkZaMo 8VgVx6n85QIvksYs8prvh57f3SFnqTP2H4ngUx8VFmopJ4lMvy4/9eQMoB/J8N5jvL6e 44Sx5hXBBH2EKOes/3khpkp7P/c+lPLuttk4leeq+UEJKAwp2yKGuks0S9hjp9P+7oBq aTfA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683058750; x=1685650750; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=R8R44OCBFHsY7LmipoXmH56AiKRTTg8SVS3Xfg7OSf0=; b=PEQjVwirUPzWbFAXnrNhnL4or2vzqV97VimJbcipaA+6ibcNfIkxJkjvaUzpxAFquH mOhQuN0Lqlx8AH2gf1kKTIv1V1h1B6ps0HIEzKRYxdjCASkiWXpXEI83wsbLaJrpgGSD D6M0Vns3beVLb+rGYQwr/x2BtRyi7InkjAXe1S2sd73/Lnnzm0qsrTxpUNyx5BKM1poR AZtL8LAVZG6HDJmAV18jw3DVJFaOmCSCbXxMXKU3a2C86DMPrEHVrrndb8MT8+kgeo0i XGAud1uoA5mJwJhKRSh0UeJofWBZj1e0XRbXa7US0B6sh2KInp1pkfM8Vyflb0/dQQiO PdTg== X-Gm-Message-State: AC+VfDwoe0HTa/L8ZlKbOlpPElxjGkvwPe/pD+Yc3It5zLF0EMA+nzDU ZWGtYooTI3fq87l0/dJms3ZETw== X-Google-Smtp-Source: ACHHUZ7ulpt92CdY0a7xC5ABJMubG8DmkB1kntwqYxi/Fw1jIGHlPkvAz+U4rnSLlKPFhN4qay81qg== X-Received: by 2002:a17:902:ec82:b0:1a1:ab92:5c88 with SMTP id x2-20020a170902ec8200b001a1ab925c88mr23193822plg.13.1683058749953; Tue, 02 May 2023 13:19:09 -0700 (PDT) Received: from [10.0.17.156] ([50.221.140.188]) by smtp.gmail.com with ESMTPSA id y5-20020a1709029b8500b0019ea9e5815bsm20232315plp.45.2023.05.02.13.19.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 02 May 2023 13:19:09 -0700 (PDT) Message-ID: <2fd806ba-30f8-8295-06a4-3e46ebeb82b2@rivosinc.com> Date: Tue, 2 May 2023 13:19:08 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: [Committed 07/11] RISC-V: Eliminate AMO op fences Content-Language: en-US To: Jeff Law , gcc-patches@gcc.gnu.org Cc: palmer@rivosinc.com, gnu-toolchain@rivosinc.com, vineetg@rivosinc.com, andrew@sifive.com, kito.cheng@sifive.com, dlustig@nvidia.com, cmuellner@gcc.gnu.org, andrea@rivosinc.com, hboehm@google.com References: <20230414170942.1695672-1-patrick@rivosinc.com> <20230427162301.1151333-1-patrick@rivosinc.com> <20230427162301.1151333-8-patrick@rivosinc.com> From: Patrick O'Neill In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-4.3 required=5.0 tests=BAYES_00,BODY_8BITS,DKIM_SIGNED,DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On 4/28/23 10:43, Jeff Law wrote: > > > On 4/27/23 10:22, Patrick O'Neill wrote: >> Atomic operations with the appropriate bits set already enfore release >> semantics. Remove unnecessary release fences from atomic ops. >> >> This change brings AMO ops in line with table A.6 of the ISA manual. >> >> 2023-04-27 Patrick O'Neill >> >> gcc/ChangeLog: >> >>     * config/riscv/riscv.cc >>     (riscv_memmodel_needs_amo_release): Change function name. >>     (riscv_print_operand): Remove unneeded %F case. >>     * config/riscv/sync.md: Remove unneeded fences. > OK.  Though note this depends on a resolution of patch #6.  You could > potentially leave the %F support in riscv_print_operand and install > the rest of this patch while we settle the question around #6. > > Jeff Committed. Patrick