From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lj1-x230.google.com (mail-lj1-x230.google.com [IPv6:2a00:1450:4864:20::230]) by sourceware.org (Postfix) with ESMTPS id 595023858C60 for ; Tue, 2 Nov 2021 11:16:31 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 595023858C60 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=sifive.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=sifive.com Received: by mail-lj1-x230.google.com with SMTP id x19so10137263ljm.11 for ; Tue, 02 Nov 2021 04:16:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sifive.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=4FVCRRLskYgqbV6Ssw4yQU85OSm7rChTzgxsYWUWVIM=; b=kKFKEqJRa7kcrnCC91sy0LS+THO8AC8HQiRri7D2f8h7WZUvuexEBR3c+nIcw9qqJU cgehSrfq3NeZxq9WXao0YN5zYQCyFBtOV01luDQUp1CvyUcTnhP4ds/aS5W3wd+jUsVK ctZIkZZ2h5T8b1LbJp8FkYt31SksUJVakw9e5iTl6GFqq3MFKKa2bkm412itWoegtMNM p75ndMmLdGrZtnH02dNGlpUfvOxTJLsgocv3IfK7OcvNiqiub4L/04elOGPFyB8VjYdw BOQ0qPO/rUleR//a1Wzk7I4bDQYvD/3XNH6HhTONq4qARZjgORU1jnPs3HsqO+9bnTY4 xddg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=4FVCRRLskYgqbV6Ssw4yQU85OSm7rChTzgxsYWUWVIM=; b=qAb8msCoyDjIUT51/Hb5uLkEAk+EnLtUZpOg/tcZVDctPFpNv/VR2cyKxUQhgg37tJ 6Ta15BITa1FRYkwMm8tu7Swg/Sc2tgXHIpnOpGEmUczzGicVufiiBjHhy2qpUzuQfN+e Hz0FImbhA/O/poY+WoED8rw7AUu+pzqqg/0u9N/GdtVr0dUDc/KlG7uV6PrkQvs/GLpl V5Bc7tqkwS/QOTiIXe0yW0/8vAvytLAVC4YKaVoo4SqP54/bu8qm3rYxMaHzMocmeY2/ gRBoqlPF+q6G7xspADsoq3HN+KfjQf9CJS3BnwawtxqWq2WxSKK9uRHq+JQfD/0Km45X u2EQ== X-Gm-Message-State: AOAM533Sr8NcMw0g76BQ5GpanJTG/sU1NLH9z8g4I8Nkh+X1ttJM7V0o pusOjjY/uMG99hiWwG/ygBkNc6FwZuUYvOHBUAsPhQ== X-Google-Smtp-Source: ABdhPJxSdXJ08UbiCPmmH2yuiODzEY0lUVeNBCiwvD4klpnse5g7LsqH0uV7u2VeDkSoZiRNNTfN61howeLMkemBzUA= X-Received: by 2002:a2e:7f1b:: with SMTP id a27mr36870946ljd.55.1635851790049; Tue, 02 Nov 2021 04:16:30 -0700 (PDT) MIME-Version: 1.0 References: <20211031093445.1414518-1-siyu@isrc.iscas.ac.cn> In-Reply-To: <20211031093445.1414518-1-siyu@isrc.iscas.ac.cn> From: Kito Cheng Date: Tue, 2 Nov 2021 19:16:19 +0800 Message-ID: Subject: Re: [PATCH 00/21] RISC-V: add gcc support for Scalar Cryptography v1.0.0-rc5 To: siyu@isrc.iscas.ac.cn Cc: GCC Patches , Jim Wilson , Christoph Muellner , Palmer Dabbelt , Andrew Waterman , =?UTF-8?B?V2VpIFd1ICjlkLTkvJ8p?= , jiawei , ben.marshall@pqshield.com, mjos@pqshield.com, Richard.Newell@microchip.com Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-3.9 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, KAM_NUMSUBJECT, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Nov 2021 11:16:33 -0000 Hi Si-Yu: Thanks for your patch! Here is a general comment for this patch-set, GCC require ChagneLog in commit log, and seems like that is included in your commit log, you can refer other commit log to see how changelog wrote. Scalar cryptography extension seems not define builtin and intrinsic within the spec, I would prefer add those builtin to riscv-c-api-doc[1] before adding to GCC for make sure LLVM and GCC toolchain has consistent interface, so I would like to merge arch string related stuffs only first. So could you send a V2 patchset without builtins and md changes? [1] https://github.com/riscv-non-isa/riscv-c-api-doc/blob/master/riscv-c-api.md Thanks! On Sun, Oct 31, 2021 at 5:34 PM wrote: > > From: SiYu Wu > > This patch add gcc backend support for RISC-V Scalar Cryptography > Extension (k-ext), including machine description, builtins defines and > testcases for each k-ext's subset. > > A note about Zbkx: The Zbkx should be implemented in bitmanip's Zbp, but > since zbp is not included in the bitmanip spec v1.0, and crypto's v1.0 > release will earlier than bitmanip's next release, so for now we > implementing it here. > > SiYu Wu (19): > [crypto]: add machine description for Zknd and Zkne > [crypto]: add builtins for Zknd and Zkne > [crypto]: add testcases for Zknd and Zkne > [crypto]: add machine description for Zknh > [crypto]: add builtins for Zknh > [crypto]: add testcases for Zknh > [crypto]: add machine description for Zksed > [crypto]: add builtins for Zksed > [crypto]: add testcases for Zksed > [crypto]: add machine description for Zksh > [crypto]: add builtins for Zksh > [crypto]: add testcases for Zksh > [crypto]: add option defines for Zkr and Zkt > [crypto]: add option defines for Zbkb, Zbkc and Zbkx > [crypto]: add implied defines of Zk, Zkn and Zks > change z* subset assert to allow "zk" > [crypto]: add machine description for Zbkx > [crypto]: add builtins for Zbkx > [crypto]: add testcases for Zbkx > > jiawei (1): > Fix attribute bugs due to zicsr/zifencei > > linsinan1995 (1): > Fix riscv_expand_block_move > > gcc/common/config/riscv/riscv-common.c | 39 ++- > gcc/config/riscv/arch-canonicalize | 18 +- > gcc/config/riscv/crypto.md | 319 ++++++++++++++++++ > gcc/config/riscv/riscv-builtins-crypto.def | 76 +++++ > gcc/config/riscv/riscv-builtins.c | 25 ++ > gcc/config/riscv/riscv-ftypes.def | 6 + > gcc/config/riscv/riscv-opts.h | 21 ++ > gcc/config/riscv/riscv.c | 2 +- > gcc/config/riscv/riscv.md | 4 +- > gcc/config/riscv/riscv.opt | 3 + > gcc/testsuite/gcc.target/riscv/Zbkx.c | 17 + > gcc/testsuite/gcc.target/riscv/Zknd-aes-01.c | 15 + > gcc/testsuite/gcc.target/riscv/Zknd-aes-02.c | 21 ++ > gcc/testsuite/gcc.target/riscv/Zkne-aes-01.c | 15 + > gcc/testsuite/gcc.target/riscv/Zkne-aes-02.c | 27 ++ > gcc/testsuite/gcc.target/riscv/Zknh-sha256.c | 27 ++ > .../gcc.target/riscv/Zknh-sha512-01.c | 40 +++ > .../gcc.target/riscv/Zknh-sha512-02.c | 28 ++ > gcc/testsuite/gcc.target/riscv/Zksed-sm4.c | 17 + > gcc/testsuite/gcc.target/riscv/Zksh-sm3.c | 15 + > 20 files changed, 730 insertions(+), 5 deletions(-) > create mode 100644 gcc/config/riscv/crypto.md > create mode 100644 gcc/config/riscv/riscv-builtins-crypto.def > create mode 100644 gcc/testsuite/gcc.target/riscv/Zbkx.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zknd-aes-01.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zknd-aes-02.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zkne-aes-01.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zkne-aes-02.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zknh-sha256.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zknh-sha512-01.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zknh-sha512-02.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zksed-sm4.c > create mode 100644 gcc/testsuite/gcc.target/riscv/Zksh-sm3.c > > -- > 2.25.1 >