From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29298 invoked by alias); 23 May 2013 03:41:47 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 29289 invoked by uid 89); 23 May 2013 03:41:46 -0000 X-Spam-SWARE-Status: No, score=-3.3 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE,SPF_PASS autolearn=ham version=3.3.1 Received: from mail-vc0-f178.google.com (HELO mail-vc0-f178.google.com) (209.85.220.178) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Thu, 23 May 2013 03:41:46 +0000 Received: by mail-vc0-f178.google.com with SMTP id id13so1830584vcb.37 for ; Wed, 22 May 2013 20:41:44 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.58.233.233 with SMTP id tz9mr4039801vec.13.1369280504321; Wed, 22 May 2013 20:41:44 -0700 (PDT) Received: by 10.220.175.131 with HTTP; Wed, 22 May 2013 20:41:44 -0700 (PDT) In-Reply-To: References: <20130520204053.GA21090@ibm-tiger.the-meissners.org> <20130520231307.GA538@ibm-tiger.the-meissners.org> Date: Thu, 23 May 2013 03:41:00 -0000 Message-ID: Subject: Re: [PATCH, rs6000] power8 patches, patch #2, add crypto builtins From: David Edelsohn To: Michael Meissner , GCC Patches , Pat Haugen , Peter Bergner Content-Type: text/plain; charset=ISO-8859-1 X-SW-Source: 2013-05/txt/msg01326.txt.bz2 Mike, When you committed the patch, you did not add the new rs6000/crypto.md file to the repository. - David On Tue, May 21, 2013 at 11:30 PM, David Edelsohn wrote: > On Mon, May 20, 2013 at 7:13 PM, Michael Meissner > wrote: >> This patch adds the builtins for the new ISA 2.07 crypto instructions. It >> bootstraps and causes no regressions, is it ok to install after patch #1 has >> been applied? >> >> [gcc] >> 2013-05-20 Michael Meissner >> >> * doc/extend.texi (PowerPC AltiVec/VSX Built-in Functions): Add >> documentation for the power8 crypto builtins. >> >> * config/rs6000/t-rs6000 (MD_INCLUDES): Add crypto.md. >> >> * config/rs6000/rs6000-builtin.def (BU_P8V_AV_1): Add support >> macros for defining power8 builtin functions. >> (BU_P8V_AV_2): Likewise. >> (BU_P8V_AV_P): Likewise. >> (BU_P8V_VSX_1): Likewise. >> (BU_P8V_OVERLOAD_1): Likewise. >> (BU_P8V_OVERLOAD_2): Likewise. >> (BU_CRYPTO_1): Likewise. >> (BU_CRYPTO_2): Likewise. >> (BU_CRYPTO_3): Likewise. >> (BU_CRYPTO_OVERLOAD_1): Likewise. >> (BU_CRYPTO_OVERLOAD_2): Likewise. >> (XSCVSPDP): Fix typo, point to the correct instruction. >> (VCIPHER): Add power8 crypto builtins. >> (VCIPHERLAST): Likewise. >> (VNCIPHER): Likewise. >> (VNCIPHERLAST): Likewise. >> (VPMSUMB): Likewise. >> (VPMSUMH): Likewise. >> (VPMSUMW): Likewise. >> (VPERMXOR_V2DI): Likewise. >> (VPERMXOR_V4SI: Likewise. >> (VPERMXOR_V8HI: Likewise. >> (VPERMXOR_V16QI: Likewise. >> (VSHASIGMAW): Likewise. >> (VSHASIGMAD): Likewise. >> (VPMSUM): Likewise. >> (VPERMXOR): Likewise. >> (VSHASIGMA): Likewise. >> >> * config/rs6000/rs6000-c.c (rs6000_target_modify_macros): Define >> __CRYPTO__ if the crypto instructions are available. >> (altivec_overloaded_builtins): Add support for overloaded power8 >> builtins. >> >> * config/rs6000/rs6000.c (rs6000_expand_ternop_builtin): Add >> support for power8 crypto builtins. >> (builtin_function_type): Likewise. >> (altivec_init_builtins): Add support for builtins that take vector >> long long (V2DI) arguments. >> >> * config/rs6000/crypto.md: New file, define power8 crypto >> instructions. >> >> [gcc/testsuite] >> 2013-05-20 Michael Meissner >> >> * gcc.target/powerpc/crypto-builtin-1.c: New file, test for power8 >> crypto builtins. > > Patch #2 is okay. > > Thanks, David