From patchwork Tue Nov 15 04:46:15 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sandra Loosemore X-Patchwork-Id: 20183 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp2518316wru; Mon, 14 Nov 2022 20:47:25 -0800 (PST) X-Google-Smtp-Source: AA0mqf5cOlY2+IOtZQtEYJ1jMsXh3P434nB/N3rzn+14Z7+SUvz0BmCaw5Pva4suxi8jw75EZMgU X-Received: by 2002:a17:906:a159:b0:7ad:d11e:a2c2 with SMTP id bu25-20020a170906a15900b007add11ea2c2mr12707456ejb.473.1668487645128; Mon, 14 Nov 2022 20:47:25 -0800 (PST) Received: from sourceware.org (server2.sourceware.org. [2620:52:3:1:0:246e:9693:128c]) by mx.google.com with ESMTPS id z17-20020a05640240d100b00461bde34a12si10248831edb.627.2022.11.14.20.47.24 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Nov 2022 20:47:25 -0800 (PST) Received-SPF: pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 2620:52:3:1:0:246e:9693:128c as permitted sender) client-ip=2620:52:3:1:0:246e:9693:128c; Authentication-Results: mx.google.com; dkim=pass header.i=@gcc.gnu.org header.s=default header.b=gOxy4Jgc; arc=fail (signature failed); spf=pass (google.com: domain of gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org designates 2620:52:3:1:0:246e:9693:128c as permitted sender) smtp.mailfrom="gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=gnu.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id AF7FB3832341 for ; Tue, 15 Nov 2022 04:47:23 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org AF7FB3832341 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1668487643; bh=fdfeTAYxi3XdDI8CjsqQMrEh8EGMOVaoavRMQ3ABI6U=; h=Date:To:Cc:Subject:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=gOxy4JgcFzin1i6zCFDo3jMKbBqbltP2R7Xl9DhfCDLbow/bwCAs0h6RWh/SLlqEk ffZV7MALJapskga9N+/3gnPg900BEbn1DXoALWjAaLWmQel6JjYer4dY7EZ9ZN0Cnr crAVR82Xu5cKMxBGBr1fZ/l0RNcSt+k+PAKkzqiQ= X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from NAM10-BN7-obe.outbound.protection.outlook.com (mail-bn7nam10on2061.outbound.protection.outlook.com [40.107.92.61]) by sourceware.org (Postfix) with ESMTPS id 61F86383234B for ; Tue, 15 Nov 2022 04:46:29 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 61F86383234B ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GuSyf5jI1GmNuJE5kgMnCU4+Ejhfh/evKVdXFUxWB/ATTsymIyg9spPhsTeDi2qW5uhkmfTDD6xiASe12V3w5qUflN49Ior5rvKI9MXVcGxtj7VTMlqtWrT2DViduADToImqx+fZoMhhHWt/kzFYoFm0Zx/ITiNdleJ68YNgX3c/N1/1Sxeloybhjb+duLhByG2abGTST9KiEPy8comMctTf4FBG9DYqa9UvYj6XVUwx7b/cxJadJfHxlrCVRHGpg4g+vrieNOzVcrS6qg4gsuagN2gsAyTc4uFgt6gNcndz6toJioiXabubHTHIDu1xzD2zNhU7Z8e/NCOEkGHZrA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=fdfeTAYxi3XdDI8CjsqQMrEh8EGMOVaoavRMQ3ABI6U=; b=Pn6sS8A/cO0mZYnz6MZTTnAWBwCLKLq8u0j99UaXFq2bGRO0409vk0aXKhV2MEla7BY1Q0m8uJqJmMSYr9t4E+ObaejQ3UMAoUAqRA42bmsF8Ouw3xS39DyExbN0UeGr1nH6dhOAFtQCB4ro6xCRxbWzWmGf4u6D85zF+UXsGEfpdegVxpGr19qj7Nv4vQw2uVuy5HfzE0sqCGoKxYH8wy80ls6iC76jNWAnhqI0IeQUZlBjQiUOTSUZB7yd8mPEtD2f1T5R5HkgnfKjk9SdSSY2YYJvE3xaqR0mhN9Cjss+lXjNx0vit4r8GFhiR9wYA9r8bXUYTl/zML7pFjBRRA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=siemens.com; dmarc=pass action=none header.from=siemens.com; dkim=pass header.d=siemens.com; arc=none Received: from CH2PR07MB7222.namprd07.prod.outlook.com (2603:10b6:610:ab::21) by SJ0PR07MB8584.namprd07.prod.outlook.com (2603:10b6:a03:371::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5813.17; Tue, 15 Nov 2022 04:46:20 +0000 Received: from CH2PR07MB7222.namprd07.prod.outlook.com ([fe80::bd66:6eab:d2d8:2643]) by CH2PR07MB7222.namprd07.prod.outlook.com ([fe80::bd66:6eab:d2d8:2643%5]) with mapi id 15.20.5813.017; Tue, 15 Nov 2022 04:46:19 +0000 Message-ID: <927ad110-065e-9414-1312-bff5a0644e97@siemens.com> Date: Mon, 14 Nov 2022 21:46:15 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Content-Language: en-US To: "gcc-patches@gcc.gnu.org" Cc: Jakub Jelinek Subject: [PATCH v4] OpenMP: Generate SIMD clones for functions with "declare target" X-ClientProxiedBy: CH2PR10CA0008.namprd10.prod.outlook.com (2603:10b6:610:4c::18) To CH2PR07MB7222.namprd07.prod.outlook.com (2603:10b6:610:ab::21) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR07MB7222:EE_|SJ0PR07MB8584:EE_ X-MS-Office365-Filtering-Correlation-Id: 9759da5e-b7a4-468b-21aa-08dac6c455a6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: G3ovZiK5OczVxv27aBAjh+25sJ0hYXq37gKMw02bNg+QC7hoaC+MX6alZsdc3OUmVDG2BVHv72As2nqjSU0uYggHmSYQhchM9Ou9mhiH6H7fzs6+woZ4BHqAOnOcU+lPteH8QlzJjWv1BGlLBrMPIu4Kd2dGNYdweky8x+7Ne9ybOTr1uBMjdAoV2XHSdq3h8vmzTjefgpGp+aE/z2ePMq37eBFYtX67SIKHDh9rbPrFJenYoVxVN00wotngve4iDlgosGG45x/n9blUbyunlZa6Lvf1u8rQZOAytbEEc61yAfB90eLTjUdy2DtfVcx8kasFbsh3alzQWkmciWiAknPNcDrm3AiwVQPTgOBiS2JfU5O/3kotz+tBNHzOaQDA71p42cqGPBLoB3kqOBmxWR3RjSKaLCebSXyW8zYtgLSN6Jx1E3YfsjvBB/3rSjjm912hDI2WtlSOs4G/oRfP0ISFjKvHEanolIll6uSRUnvxePnXIjlxa6uQ46b4D2C8RDAYJCmOwPMbTV27WRd3o754fs0WjZPpE9UE2v+xVbOE/7q4RmXAKUMmTdcMW6b1mM9oQeXLt8hEc8H0eFOgtNdzwAfxS93ELSrlqVCmxEhgM+gt2I8kgl5d92gpQy8nVT43MCXbgftbSRyJRScTo+pb6IpM0iVWtuImPdFy+case+dzHvVxXbl2HlHLHy5TwsqY0BRxl5GXsFlgpgjyePMLXdJyRW8C8vWito6dHd9Ht8NPoLYTEs4cJ6ZLuhdUsfpNp5CflNkV20+xSQZpRjToKJ0+XiLDiVcu4vCGbS0= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR07MB7222.namprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(4636009)(396003)(376002)(346002)(366004)(39860400002)(136003)(451199015)(36756003)(86362001)(31696002)(33964004)(6506007)(6512007)(6666004)(2616005)(186003)(82960400001)(38100700002)(8936002)(44832011)(4326008)(66476007)(66556008)(66946007)(8676002)(235185007)(5660300002)(41300700001)(2906002)(6486002)(478600001)(316002)(6916009)(31686004)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?T53REJmaYSczmFT/Idwo6gI8B+3R?= =?utf-8?q?iUSpifvMdBGn9sSoe90Bmu1lbY2HRvUZiEvPUXdVFEPmXZyxv5xRzuL6FRS6ydzka?= =?utf-8?q?QmeROqX3ChJ7t4iPcAcJ8Gqz59cTY7HhMhWRT4FhkALDPgsG3531c+OMWPWxqFFSB?= =?utf-8?q?xmQBOR161F7FK0QZ7BzQ43zKpDJkcX9oY+EqVJW3hvXRb1WGGXFWhLb7hiwbiOX6Y?= =?utf-8?q?VxaBB9hUJSKLtrJxcDUY7Gt7GTLvlOMhOiJtrMMU9LRS/DMqCpJxkhG0CDCh8fM1S?= =?utf-8?q?E5akQ56iGyQtIj+CHzeDiZuE8d/49T98DOsvK8yDHyQf7rGriJPCUeO9p2l9YrgrZ?= =?utf-8?q?mrogipenK/4UIN6eGxsnF3PoJ8XGWBDBTo2G4XBI+q9hZaCB/OyY3lUJ6VMqH6TA0?= =?utf-8?q?tYtzvV2hJbG8pdw63YyHouwf/mEMUwtSomxltNpKe9N3Vfq7UY+CZ1DGb/hD7K3nX?= =?utf-8?q?DOiK3jzwuYZD5ZkCObiaVmG5OUwxG9BjB3h6KGmHEGN/kCPm4Cs4o4FhLpVXNw6N6?= =?utf-8?q?Kqdy1vbTe2O/zeXgEU/sFtsKoMqIfwsoS7DcnfBi2L2p5vEi0STx+XA3MMa+4bhK+?= =?utf-8?q?8q/rw3hfk4vj0GczqqymafHb+bnaPNWVHQ3ngNLLEh5WORf/4R80kKqQyrAtixba6?= =?utf-8?q?swC0Ki4CiKvfRBfziitWIQKvBShwyYjSqyLaZP4c70EpHBCPGHr8bw+DSLpbZJbJ3?= =?utf-8?q?5zkovVTZ6OmLjKykrDpdQSOhX7ItsqfyuM0yBa+wrjRuMyzcqvcyJFLXVCHh21DPz?= =?utf-8?q?zss6tMBfQlV/mFCTrdiY5H1TOCRzjNN9KpgtrN/XeWDc1kFoBp6jiQzqEDwztnzta?= =?utf-8?q?Ks506gDhhtjpD+iyDCbueLXImGLUCon2xsYsALxkIGpQbvJjKIhHdLTLOma690bH0?= =?utf-8?q?x0pOioTxebgnUAA9osJhTOxMIE+HbjAljdprOLyuft6aBPB6inF42edYtdyivlNcM?= =?utf-8?q?tp8fPFvIZZUmr8HCKVHF3Jz4/DMeb+jO1exFnXidQHRT7RJ//6+YVlnne222jc2V7?= =?utf-8?q?ZWRVlzg0KpLJoneGALam4A4wKUr5fsGrnmOFoN9wxcGvxSOJazyGr0OZcFontDDvr?= =?utf-8?q?rjJ92JPHvB4Nc0oKRe+eaw9xczuURBBUl423iVLmDAbtjs2nzpVgMu4Ifrc4sSkTn?= =?utf-8?q?xaSIzmPIxXL0KlUDgAIIJHuFEO89+L0reJVOSRdR9tH2jqtdB9H4fdNQGnwb1NYwn?= =?utf-8?q?axTZ7AkFXEyarivdjdKRypIkUnozD4aJPuOY7J40CXYkplRY95qVh7CjpIxEOpbSb?= =?utf-8?q?rkn9WiTQg44HNkWVzFFDB82ygDn+U4wAOsdTmb/+e4PxrnGUJG5zyeVepmoYbdYn+?= =?utf-8?q?4yd/M6Bb9D/fFX4HN+w7Xl62wL2oj2mflwt6IoazO2dzMNl1l/3abkXRSUnm8zkOq?= =?utf-8?q?X1ajXr8UVTnHU9vrIn6TTac/eXdhWzV5dKal6LxboF1yAKQMq0FHfdXLfYcXEUARk?= =?utf-8?q?4JWIl5bJcafjqgOBrPWcQ1TZxrPgxO28oT15DrXwn45/fu5t8tw3vQMrSpSXYW5Qo?= =?utf-8?q?qzS2/Dsd5AruSDOCBsfG0xB3mmqnTpcg17tODm73KJvP0Xn3nO3eyKOpOEYKPrbsN?= =?utf-8?q?4tt9/3116SD0BF1UGiZbkU7ldr/r5KrIQ=3D=3D?= X-OriginatorOrg: siemens.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9759da5e-b7a4-468b-21aa-08dac6c455a6 X-MS-Exchange-CrossTenant-AuthSource: CH2PR07MB7222.namprd07.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Nov 2022 04:46:19.7947 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: ECwwPx3Am+cJbg9WCqiPmfgK4QsA7tzuTTDFEFWXN5FPpQKaIIwPGbikZkICTtagixkh+HaUgliuk6GJ1bIy9TOb977fKhZ4U9igU5Wpa8U= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR07MB8584 X-Spam-Status: No, score=-12.9 required=5.0 tests=BAYES_00, DKIMWL_WL_MED, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FORGED_SPF_HELO, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_PASS, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) 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-Patchwork-Original-From: Sandra Loosemore via Gcc-patches From: Sandra Loosemore Reply-To: Sandra Loosemore Errors-To: gcc-patches-bounces+ouuuleilei=gmail.com@gcc.gnu.org Sender: "Gcc-patches" X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1749536100568016003?= X-GMAIL-MSGID: =?utf-8?q?1749536100568016003?= Here is yet another attempt at a patch to auto-generate SIMD clones for functions that already have the "declare target" attribute. This version v4 is derived from the previous v2 version, since v3 seemed to be a dead end. I have added conditionals to restrict the auto-generation at -O2 to the offload compiler, and extended the syntax of the -fopenmp-target-simd-clone to allow explicit control over whether it applies to host, target, or both -- this primarily to allow better test coverage. I've added infrastructure to support testing on the offload compiler, added new test cases, and reworked the existing test cases to scan for interesting things written to the dump file instead of examining the .s output. I hope it is not too late to consider this patch given that I've been trying to get this feature in for months already. Also, I kind of got caught in the Sphinx churn last week, relating to the documentation parts of this patch. :-( I understand that if this patch is accepted I am also on the hook to come up with a further patch to try to GC unused clones after vectorization; I haven't started on that piece yet. -Sandra From 771be96d2dc7b8868ba06cf8ec6afe7a3337ac89 Mon Sep 17 00:00:00 2001 From: Sandra Loosemore Date: Tue, 15 Nov 2022 03:40:12 +0000 Subject: [PATCH] OpenMP: Generate SIMD clones for functions with "declare target" This patch causes the IPA simdclone pass to generate clones for functions with the "omp declare target" attribute as if they had "omp declare simd", provided the function appears to be suitable for SIMD execution. The filter is conservative, rejecting functions that write memory or that call other functions not known to be safe. A new option -fopenmp-target-simd-clone is added to control this transformation; it's enabled for offload processing at -O2 and higher. gcc/ChangeLog: * common.opt (fopenmp-target-simd-clone): New option. (target_simd_clone_device): New enum to go with it. * doc/invoke.texi (-fopenmp-target-simd-clone): Document. * flag-types.h (enum omp_target_simd_clone_device_kind): New. * omp-simd-clone.cc (auto_simd_fail): New function. (auto_simd_check_stmt): New function. (plausible_type_for_simd_clone): New function. (ok_for_auto_simd_clone): New function. (simd_clone_create): Add force_local argument, make the symbol have internal linkage if it is true. (expand_simd_clones): Also check for cloneable functions with "omp declare target". Pass explicit_p argument to simd_clone.compute_vecsize_and_simdlen target hook. * opts.cc (default_options_table): Add -fopenmp-target-simd-clone. * target.def (TARGET_SIMD_CLONE_COMPUTE_VECSIZE_AND_SIMDLEN): Add bool explicit_p argument. * doc/tm.texi: Regenerated. * config/aarch64/aarch64.cc (aarch64_simd_clone_compute_vecsize_and_simdlen): Update. * config/gcn/gcn.cc (gcn_simd_clone_compute_vecsize_and_simdlen): Update. * config/i386/i386.cc (ix86_simd_clone_compute_vecsize_and_simdlen): Update. gcc/testsuite/ChangeLog: * g++.dg/gomp/target-simd-clone-1.C: New. * g++.dg/gomp/target-simd-clone-2.C: New. * gcc.dg/gomp/target-simd-clone-1.c: New. * gcc.dg/gomp/target-simd-clone-2.c: New. * gcc.dg/gomp/target-simd-clone-3.c: New. * gcc.dg/gomp/target-simd-clone-4.c: New. * gcc.dg/gomp/target-simd-clone-5.c: New. * gcc.dg/gomp/target-simd-clone-6.c: New. * gcc.dg/gomp/target-simd-clone-7.c: New. * gcc.dg/gomp/target-simd-clone-8.c: New. * lib/scanoffloadipa.exp: New. libgomp/ChangeLog: * testsuite/lib/libgomp.exp: Load scanoffloadipa.exp library. * testsuite/libgomp.c/target-simd-clone-1.c: New. * testsuite/libgomp.c/target-simd-clone-2.c: New. * testsuite/libgomp.c/target-simd-clone-3.c: New. --- gcc/common.opt | 22 ++ gcc/config/aarch64/aarch64.cc | 24 +- gcc/config/gcn/gcn.cc | 10 +- gcc/config/i386/i386.cc | 27 +- gcc/doc/invoke.texi | 23 +- gcc/doc/tm.texi | 2 +- gcc/flag-types.h | 9 + gcc/omp-simd-clone.cc | 309 ++++++++++++++++-- gcc/opts.cc | 2 + gcc/target.def | 2 +- .../g++.dg/gomp/target-simd-clone-1.C | 25 ++ .../g++.dg/gomp/target-simd-clone-2.C | 23 ++ .../gcc.dg/gomp/target-simd-clone-1.c | 25 ++ .../gcc.dg/gomp/target-simd-clone-2.c | 22 ++ .../gcc.dg/gomp/target-simd-clone-3.c | 22 ++ .../gcc.dg/gomp/target-simd-clone-4.c | 26 ++ .../gcc.dg/gomp/target-simd-clone-5.c | 28 ++ .../gcc.dg/gomp/target-simd-clone-6.c | 27 ++ .../gcc.dg/gomp/target-simd-clone-7.c | 15 + .../gcc.dg/gomp/target-simd-clone-8.c | 25 ++ gcc/testsuite/lib/scanoffloadipa.exp | 148 +++++++++ libgomp/testsuite/lib/libgomp.exp | 1 + .../testsuite/libgomp.c/target-simd-clone-1.c | 43 +++ .../testsuite/libgomp.c/target-simd-clone-2.c | 39 +++ .../testsuite/libgomp.c/target-simd-clone-3.c | 40 +++ 25 files changed, 887 insertions(+), 52 deletions(-) create mode 100644 gcc/testsuite/g++.dg/gomp/target-simd-clone-1.C create mode 100644 gcc/testsuite/g++.dg/gomp/target-simd-clone-2.C create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-1.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-2.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-3.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-4.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-5.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-6.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-7.c create mode 100644 gcc/testsuite/gcc.dg/gomp/target-simd-clone-8.c create mode 100644 gcc/testsuite/lib/scanoffloadipa.exp create mode 100644 libgomp/testsuite/libgomp.c/target-simd-clone-1.c create mode 100644 libgomp/testsuite/libgomp.c/target-simd-clone-2.c create mode 100644 libgomp/testsuite/libgomp.c/target-simd-clone-3.c diff --git a/gcc/common.opt b/gcc/common.opt index 26e9d1cc4e7..c458b71680c 100644 --- a/gcc/common.opt +++ b/gcc/common.opt @@ -2218,6 +2218,28 @@ fomit-frame-pointer Common Var(flag_omit_frame_pointer) Optimization When possible do not generate stack frames. +fopenmp-target-simd-clone +Common Alias(fopenmp-target-simd-clone=,any,none) + +fopenmp-target-simd-clone= +Common Joined RejectNegative Enum(target_simd_clone_device) Var(flag_openmp_target_simd_clone) Init(OMP_TARGET_SIMD_CLONE_NONE) Optimization +Generate SIMD clones for functions with the OpenMP declare target directive. + +Enum +Name(target_simd_clone_device) Type(int) + +EnumValue +Enum(target_simd_clone_device) String(none) Value(OMP_TARGET_SIMD_CLONE_NONE) + +EnumValue +Enum(target_simd_clone_device) String(host) Value(OMP_TARGET_SIMD_CLONE_HOST) + +EnumValue +Enum(target_simd_clone_device) String(nohost) Value(OMP_TARGET_SIMD_CLONE_NOHOST) + +EnumValue +Enum(target_simd_clone_device) String(any) Value(OMP_TARGET_SIMD_CLONE_ANY) + fopt-info Common Var(flag_opt_info) Optimization Enable all optimization info dumps on stderr. diff --git a/gcc/config/aarch64/aarch64.cc b/gcc/config/aarch64/aarch64.cc index c91df6f5006..58f31835fe7 100644 --- a/gcc/config/aarch64/aarch64.cc +++ b/gcc/config/aarch64/aarch64.cc @@ -26832,7 +26832,8 @@ currently_supported_simd_type (tree t, tree b) static int aarch64_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, struct cgraph_simd_clone *clonei, - tree base_type, int num) + tree base_type, int num, + bool explicit_p) { tree t, ret_type; unsigned int elt_bits, count; @@ -26850,8 +26851,9 @@ aarch64_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, || const_simdlen > 1024 || (const_simdlen & (const_simdlen - 1)) != 0)) { - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "unsupported simdlen %wd", const_simdlen); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "unsupported simdlen %wd", const_simdlen); return 0; } @@ -26859,7 +26861,9 @@ aarch64_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, if (TREE_CODE (ret_type) != VOID_TYPE && !currently_supported_simd_type (ret_type, base_type)) { - if (TYPE_SIZE (ret_type) != TYPE_SIZE (base_type)) + if (!explicit_p) + ; + else if (TYPE_SIZE (ret_type) != TYPE_SIZE (base_type)) warning_at (DECL_SOURCE_LOCATION (node->decl), 0, "GCC does not currently support mixed size types " "for % functions"); @@ -26886,7 +26890,9 @@ aarch64_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, if (clonei->args[i].arg_type != SIMD_CLONE_ARG_TYPE_UNIFORM && !currently_supported_simd_type (arg_type, base_type)) { - if (TYPE_SIZE (arg_type) != TYPE_SIZE (base_type)) + if (!explicit_p) + ; + else if (TYPE_SIZE (arg_type) != TYPE_SIZE (base_type)) warning_at (DECL_SOURCE_LOCATION (node->decl), 0, "GCC does not currently support mixed size types " "for % functions"); @@ -26916,9 +26922,11 @@ aarch64_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, if (clonei->simdlen.is_constant (&const_simdlen) && maybe_ne (vec_bits, 64U) && maybe_ne (vec_bits, 128U)) { - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "GCC does not currently support simdlen %wd for type %qT", - const_simdlen, base_type); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "GCC does not currently support simdlen %wd for " + "type %qT", + const_simdlen, base_type); return 0; } } diff --git a/gcc/config/gcn/gcn.cc b/gcc/config/gcn/gcn.cc index 5e6f3b8b74b..4b9eeeeaf84 100644 --- a/gcc/config/gcn/gcn.cc +++ b/gcc/config/gcn/gcn.cc @@ -5061,7 +5061,8 @@ static int gcn_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *ARG_UNUSED (node), struct cgraph_simd_clone *clonei, tree ARG_UNUSED (base_type), - int ARG_UNUSED (num)) + int ARG_UNUSED (num), + bool explicit_p) { if (known_eq (clonei->simdlen, 0U)) clonei->simdlen = 64; @@ -5069,9 +5070,10 @@ gcn_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *ARG_UNUSED (node { /* Note that x86 has a similar message that is likely to trigger on sizes that are OK for gcn; the user can't win. */ - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "unsupported simdlen %wd (amdgcn)", - clonei->simdlen.to_constant ()); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "unsupported simdlen %wd (amdgcn)", + clonei->simdlen.to_constant ()); return 0; } diff --git a/gcc/config/i386/i386.cc b/gcc/config/i386/i386.cc index 292b32c5e99..abd1d008776 100644 --- a/gcc/config/i386/i386.cc +++ b/gcc/config/i386/i386.cc @@ -23633,7 +23633,8 @@ ix86_memmodel_check (unsigned HOST_WIDE_INT val) static int ix86_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, struct cgraph_simd_clone *clonei, - tree base_type, int num) + tree base_type, int num, + bool explicit_p) { int ret = 1; @@ -23642,8 +23643,9 @@ ix86_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, || clonei->simdlen > 1024 || (clonei->simdlen & (clonei->simdlen - 1)) != 0)) { - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "unsupported simdlen %wd", clonei->simdlen.to_constant ()); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "unsupported simdlen %wd", clonei->simdlen.to_constant ()); return 0; } @@ -23663,8 +23665,9 @@ ix86_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, break; /* FALLTHRU */ default: - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "unsupported return type %qT for simd", ret_type); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "unsupported return type %qT for simd", ret_type); return 0; } @@ -23693,13 +23696,14 @@ ix86_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, default: if (clonei->args[i].arg_type == SIMD_CLONE_ARG_TYPE_UNIFORM) break; - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "unsupported argument type %qT for simd", arg_type); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "unsupported argument type %qT for simd", arg_type); return 0; } } - if (!TREE_PUBLIC (node->decl)) + if (!TREE_PUBLIC (node->decl) || !explicit_p) { /* If the function isn't exported, we can pick up just one ISA for the clones. */ @@ -23770,9 +23774,10 @@ ix86_simd_clone_compute_vecsize_and_simdlen (struct cgraph_node *node, cnt /= clonei->vecsize_float; if (cnt > (TARGET_64BIT ? 16 : 8)) { - warning_at (DECL_SOURCE_LOCATION (node->decl), 0, - "unsupported simdlen %wd", - clonei->simdlen.to_constant ()); + if (explicit_p) + warning_at (DECL_SOURCE_LOCATION (node->decl), 0, + "unsupported simdlen %wd", + clonei->simdlen.to_constant ()); return 0; } } diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index dc2da464ebb..7c8a79ec636 100644 --- a/gcc/doc/invoke.texi +++ b/gcc/doc/invoke.texi @@ -204,7 +204,7 @@ in the following sections. -flax-vector-conversions -fms-extensions @gol -foffload=@var{arg} -foffload-options=@var{arg} @gol -fopenacc -fopenacc-dim=@var{geom} @gol --fopenmp -fopenmp-simd @gol +-fopenmp -fopenmp-simd -fopenmp-target-simd-clone@r{[}=@var{device-type}@r{]} @gol -fpermitted-flt-eval-methods=@var{standard} @gol -fplan9-extensions -fsigned-bitfields -funsigned-bitfields @gol -fsigned-char -funsigned-char -fstrict-flex-arrays[=@var{n}] @gol @@ -2758,6 +2758,27 @@ Enable handling of OpenMP's @code{simd}, @code{declare simd}, @code{[[omp::directive(...)]]} and @code{[[omp::sequence(...)]]} in C++ and @code{!$omp} in Fortran. Other OpenMP directives are ignored. +@item -fopenmp-target-simd-clone +@item -fopenmp-target-simd-clone=@var{device-type} +@opindex fopenmp-target-simd-clone +@cindex OpenMP target SIMD clone +In addition to generating SIMD clones for functions marked with the +@code{declare simd} directive, GCC also generates clones +for functions marked with the OpenMP @code{declare target} directive +that are suitable for vectorization when this option is in effect. The +@var{device-type} may be one of @code{none}, @code{host}, @code{nohost}, +and @code{any}, which correspond to keywords for the @code{device_type} +clause of the @code{declare target} directive; clones are generated for +the intersection of devices specified. +@option{-fopenmp-target-simd-clone} is equivalent to +@option{-fopenmp-target-simd-clone=any} and +@option{-fno-openmp-target-simd-clone} is equivalent to +@option{-fopenmp-target-simd-clone=none}. + +At @option{-O2} and higher (but not @option{-Os} or @option{-Og}) this +optimization defaults to @option{-fopenmp-target-simd-clone=nohost}; otherwise +it is disabled by default. + @item -fpermitted-flt-eval-methods=@var{style} @opindex fpermitted-flt-eval-methods @opindex fpermitted-flt-eval-methods=c11 diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi index af77d16030c..8fe49c2ba3d 100644 --- a/gcc/doc/tm.texi +++ b/gcc/doc/tm.texi @@ -6290,7 +6290,7 @@ The default is @code{NULL_TREE} which means to not vectorize scatter stores. @end deftypefn -@deftypefn {Target Hook} int TARGET_SIMD_CLONE_COMPUTE_VECSIZE_AND_SIMDLEN (struct cgraph_node *@var{}, struct cgraph_simd_clone *@var{}, @var{tree}, @var{int}) +@deftypefn {Target Hook} int TARGET_SIMD_CLONE_COMPUTE_VECSIZE_AND_SIMDLEN (struct cgraph_node *@var{}, struct cgraph_simd_clone *@var{}, @var{tree}, @var{int}, @var{bool}) This hook should set @var{vecsize_mangle}, @var{vecsize_int}, @var{vecsize_float} fields in @var{simd_clone} structure pointed by @var{clone_info} argument and also @var{simdlen} field if it was previously 0. diff --git a/gcc/flag-types.h b/gcc/flag-types.h index d2e751060ff..60dff0a38e9 100644 --- a/gcc/flag-types.h +++ b/gcc/flag-types.h @@ -494,6 +494,15 @@ enum openacc_privatization OPENACC_PRIVATIZATION_NOISY }; +/* Targets for -fopenmp-target-simd-clone. */ +enum omp_target_simd_clone_device_kind +{ + OMP_TARGET_SIMD_CLONE_NONE = 0, + OMP_TARGET_SIMD_CLONE_HOST = 1, + OMP_TARGET_SIMD_CLONE_NOHOST = 2, + OMP_TARGET_SIMD_CLONE_ANY = 3 +}; + #endif #endif /* ! GCC_FLAG_TYPES_H */ diff --git a/gcc/omp-simd-clone.cc b/gcc/omp-simd-clone.cc index 34cbee5afcd..21d69aa8747 100644 --- a/gcc/omp-simd-clone.cc +++ b/gcc/omp-simd-clone.cc @@ -51,6 +51,210 @@ along with GCC; see the file COPYING3. If not see #include "stringpool.h" #include "attribs.h" #include "omp-simd-clone.h" +#include "omp-low.h" +#include "omp-general.h" + +/* Print debug info for ok_for_auto_simd_clone to the dump file, logging + failure reason EXCUSE for function DECL. Always returns false. */ +static bool +auto_simd_fail (tree decl, const char *excuse) +{ + if (dump_file && (dump_flags & TDF_DETAILS)) + fprintf (dump_file, "\nNot auto-cloning %s because %s\n", + IDENTIFIER_POINTER (DECL_ASSEMBLER_NAME (decl)), + excuse); + return false; +} + +/* Helper function for ok_for_auto_simd_clone; return false if the statement + violates restrictions for an "omp declare simd" function. Specifically, + the function must not + - throw or call setjmp/longjmp + - write memory that could alias parallel calls + - read volatile memory + - include openmp directives or calls + - call functions that might do those things */ + +static bool +auto_simd_check_stmt (gimple *stmt, tree outer) +{ + tree decl; + + switch (gimple_code (stmt)) + { + case GIMPLE_CALL: + + /* Calls to functions that are CONST or PURE are ok, even if they + are internal functions without a decl. Reject other internal + functions. */ + if (gimple_call_flags (stmt) & (ECF_CONST | ECF_PURE)) + break; + if (gimple_call_internal_p (stmt)) + return auto_simd_fail (outer, + "body contains internal function call"); + + decl = gimple_call_fndecl (stmt); + + /* We can't know whether indirect calls are safe. */ + if (decl == NULL_TREE) + return auto_simd_fail (outer, "body contains indirect call"); + + /* Calls to functions that are already marked "omp declare simd" are + OK. */ + if (lookup_attribute ("omp declare simd", DECL_ATTRIBUTES (decl))) + break; + + /* Let recursive calls to the current function through. */ + if (decl == outer) + break; + + /* Other function calls are not permitted. This covers all calls to + the libgomp API and setjmp/longjmp, too, as well as things like + __cxa_throw_ related to exception handling. */ + return auto_simd_fail (outer, "body contains unsafe function call"); + + /* Reject EH-related constructs. Most of the EH gimple codes are + already lowered by the time this pass runs during IPA. + GIMPLE_EH_DISPATCH and GIMPLE_RESX remain and are lowered by + pass_lower_eh_dispatch and pass_lower_resx, respectively; those + passes run later. */ + case GIMPLE_EH_DISPATCH: + case GIMPLE_RESX: + return auto_simd_fail (outer, "body contains EH constructs"); + + /* Asms are not permitted since we don't know what they do. */ + case GIMPLE_ASM: + return auto_simd_fail (outer, "body contains inline asm"); + + default: + break; + } + + /* Memory writes are not permitted. + FIXME: this could be relaxed a little to permit writes to + function-local variables that could not alias other instances + of the function running in parallel. */ + if (gimple_store_p (stmt)) + return auto_simd_fail (outer, "body includes memory write"); + + /* Volatile reads are not permitted. */ + if (gimple_has_volatile_ops (stmt)) + return auto_simd_fail (outer, "body includes volatile op"); + + /* Otherwise OK. */ + return true; +} + +/* Helper function for ok_for_auto_simd_clone: return true if type T is + plausible for a cloneable function argument or return type. */ +static bool +plausible_type_for_simd_clone (tree t) +{ + if (TREE_CODE (t) == VOID_TYPE) + return true; + else if (RECORD_OR_UNION_TYPE_P (t) || !is_a (TYPE_MODE (t))) + /* Small record/union types may fit into a scalar mode, but are + still not suitable. */ + return false; + else if (TYPE_ATOMIC (t)) + /* Atomic types trigger warnings in simd_clone_clauses_extract. */ + return false; + else + return true; +} + +/* Check if the function NODE appears suitable for auto-annotation + with "declare simd". */ + +static bool +ok_for_auto_simd_clone (struct cgraph_node *node) +{ + tree decl = node->decl; + tree t; + basic_block bb; + + /* Nothing to do if the function isn't a definition or doesn't + have a body. */ + if (!node->definition || !node->has_gimple_body_p ()) + return auto_simd_fail (decl, "no definition or body"); + + /* No point in trying to generate implicit clones if the function + isn't used in the compilation unit. */ + if (!node->callers) + return auto_simd_fail (decl, "function is not used"); + + /* Nothing to do if the function already has the "omp declare simd" + attribute, is marked noclone, or is not "omp declare target". */ + if (lookup_attribute ("omp declare simd", DECL_ATTRIBUTES (decl)) + || lookup_attribute ("noclone", DECL_ATTRIBUTES (decl)) + || !lookup_attribute ("omp declare target", DECL_ATTRIBUTES (decl))) + return auto_simd_fail (decl, "incompatible attributes"); + + /* Check whether the function is restricted host/nohost via the + "omp declare target device_type" clause, and that doesn't match + what we're compiling for. Internally, these translate into + "omp declare target [no]host" attributes on the decl; "any" + translates into both attributes, but the default (which is supposed + to be equivalent to "any") is neither. */ + tree host = lookup_attribute ("omp declare target host", + DECL_ATTRIBUTES (decl)); + tree nohost = lookup_attribute ("omp declare target nohost", + DECL_ATTRIBUTES (decl)); +#ifdef ACCEL_COMPILER + if (host && !nohost) + return auto_simd_fail (decl, "device doesn't match for accel compiler"); +#else + if (nohost && !host) + return auto_simd_fail (decl, "device doesn't match for host compiler"); +#endif + + /* Backends will check for vectorizable arguments/return types in a + target-specific way, but we can immediately filter out functions + that have implausible argument/return types. */ + t = TREE_TYPE (TREE_TYPE (decl)); + if (!plausible_type_for_simd_clone (t)) + return auto_simd_fail (decl, "return type fails sniff test"); + + if (TYPE_ARG_TYPES (TREE_TYPE (decl))) + { + for (tree temp = TYPE_ARG_TYPES (TREE_TYPE (decl)); + temp; temp = TREE_CHAIN (temp)) + { + t = TREE_VALUE (temp); + if (!plausible_type_for_simd_clone (t)) + return auto_simd_fail (decl, "argument type fails sniff test"); + } + } + else if (DECL_ARGUMENTS (decl)) + { + for (tree temp = DECL_ARGUMENTS (decl); temp; temp = DECL_CHAIN (temp)) + { + t = TREE_TYPE (temp); + if (!plausible_type_for_simd_clone (t)) + return auto_simd_fail (decl, "argument type fails sniff test"); + } + } + else + return auto_simd_fail (decl, "function has no arguments"); + + /* Scan the function body to see if it is suitable for SIMD-ization. */ + node->get_body (); + + FOR_EACH_BB_FN (bb, DECL_STRUCT_FUNCTION (decl)) + { + for (gimple_stmt_iterator gsi = gsi_start_bb (bb); !gsi_end_p (gsi); + gsi_next (&gsi)) + if (!auto_simd_check_stmt (gsi_stmt (gsi), decl)) + return false; + } + + /* All is good. */ + if (dump_file) + fprintf (dump_file, "\nMarking %s for auto-cloning\n", + IDENTIFIER_POINTER (DECL_ASSEMBLER_NAME (decl))); + return true; +} + /* Return the number of elements in vector type VECTYPE, which is associated with a SIMD clone. At present these always have a constant length. */ @@ -430,10 +634,12 @@ simd_clone_mangle (struct cgraph_node *node, return get_identifier (str); } -/* Create a simd clone of OLD_NODE and return it. */ +/* Create a simd clone of OLD_NODE and return it. If FORCE_LOCAL is true, + create it as a local symbol, otherwise copy the symbol linkage and + visibility attributes from OLD_NODE. */ static struct cgraph_node * -simd_clone_create (struct cgraph_node *old_node) +simd_clone_create (struct cgraph_node *old_node, bool force_local) { struct cgraph_node *new_node; if (old_node->definition) @@ -463,23 +669,38 @@ simd_clone_create (struct cgraph_node *old_node) return new_node; set_decl_built_in_function (new_node->decl, NOT_BUILT_IN, 0); - TREE_PUBLIC (new_node->decl) = TREE_PUBLIC (old_node->decl); - DECL_COMDAT (new_node->decl) = DECL_COMDAT (old_node->decl); - DECL_WEAK (new_node->decl) = DECL_WEAK (old_node->decl); - DECL_EXTERNAL (new_node->decl) = DECL_EXTERNAL (old_node->decl); - DECL_VISIBILITY_SPECIFIED (new_node->decl) - = DECL_VISIBILITY_SPECIFIED (old_node->decl); - DECL_VISIBILITY (new_node->decl) = DECL_VISIBILITY (old_node->decl); - DECL_DLLIMPORT_P (new_node->decl) = DECL_DLLIMPORT_P (old_node->decl); - if (DECL_ONE_ONLY (old_node->decl)) - make_decl_one_only (new_node->decl, DECL_ASSEMBLER_NAME (new_node->decl)); - - /* The method cgraph_version_clone_with_body () will force the new - symbol local. Undo this, and inherit external visibility from - the old node. */ - new_node->local = old_node->local; - new_node->externally_visible = old_node->externally_visible; - new_node->calls_declare_variant_alt = old_node->calls_declare_variant_alt; + if (force_local) + { + TREE_PUBLIC (new_node->decl) = 0; + DECL_COMDAT (new_node->decl) = 0; + DECL_WEAK (new_node->decl) = 0; + DECL_EXTERNAL (new_node->decl) = 0; + DECL_VISIBILITY_SPECIFIED (new_node->decl) = 0; + DECL_VISIBILITY (new_node->decl) = VISIBILITY_DEFAULT; + DECL_DLLIMPORT_P (new_node->decl) = 0; + } + else + { + TREE_PUBLIC (new_node->decl) = TREE_PUBLIC (old_node->decl); + DECL_COMDAT (new_node->decl) = DECL_COMDAT (old_node->decl); + DECL_WEAK (new_node->decl) = DECL_WEAK (old_node->decl); + DECL_EXTERNAL (new_node->decl) = DECL_EXTERNAL (old_node->decl); + DECL_VISIBILITY_SPECIFIED (new_node->decl) + = DECL_VISIBILITY_SPECIFIED (old_node->decl); + DECL_VISIBILITY (new_node->decl) = DECL_VISIBILITY (old_node->decl); + DECL_DLLIMPORT_P (new_node->decl) = DECL_DLLIMPORT_P (old_node->decl); + if (DECL_ONE_ONLY (old_node->decl)) + make_decl_one_only (new_node->decl, + DECL_ASSEMBLER_NAME (new_node->decl)); + + /* The method cgraph_version_clone_with_body () will force the new + symbol local. Undo this, and inherit external visibility from + the old node. */ + new_node->local = old_node->local; + new_node->externally_visible = old_node->externally_visible; + new_node->calls_declare_variant_alt + = old_node->calls_declare_variant_alt; + } return new_node; } @@ -1683,13 +1904,42 @@ simd_clone_adjust (struct cgraph_node *node) void expand_simd_clones (struct cgraph_node *node) { - tree attr = lookup_attribute ("omp declare simd", - DECL_ATTRIBUTES (node->decl)); - if (attr == NULL_TREE - || node->inlined_to + tree attr; + bool explicit_p = true; + + if (node->inlined_to || lookup_attribute ("noclone", DECL_ATTRIBUTES (node->decl))) return; + attr = lookup_attribute ("omp declare simd", + DECL_ATTRIBUTES (node->decl)); + + /* See if we can add an "omp declare simd" directive implicitly + before giving up. */ + /* FIXME: OpenACC "#pragma acc routine" translates into + "omp declare target", but appears also to have some other effects + that conflict with generating SIMD clones, causing ICEs. So don't + do this if we've got OpenACC instead of OpenMP. */ + if (attr == NULL_TREE +#ifdef ACCEL_COMPILER + && (flag_openmp_target_simd_clone == OMP_TARGET_SIMD_CLONE_ANY + || flag_openmp_target_simd_clone == OMP_TARGET_SIMD_CLONE_NOHOST) +#else + && (flag_openmp_target_simd_clone == OMP_TARGET_SIMD_CLONE_ANY + || flag_openmp_target_simd_clone == OMP_TARGET_SIMD_CLONE_HOST) +#endif + && !oacc_get_fn_attrib (node->decl) + && ok_for_auto_simd_clone (node)) + { + attr = tree_cons (get_identifier ("omp declare simd"), NULL, + DECL_ATTRIBUTES (node->decl)); + DECL_ATTRIBUTES (node->decl) = attr; + explicit_p = false; + } + + if (attr == NULL_TREE) + return; + /* Ignore #pragma omp declare simd extern int foo (); @@ -1714,13 +1964,15 @@ expand_simd_clones (struct cgraph_node *node) poly_uint64 orig_simdlen = clone_info->simdlen; tree base_type = simd_clone_compute_base_data_type (node, clone_info); + /* The target can return 0 (no simd clones should be created), 1 (just one ISA of simd clones should be created) or higher count of ISA variants. In that case, clone_info is initialized for the first ISA variant. */ int count = targetm.simd_clone.compute_vecsize_and_simdlen (node, clone_info, - base_type, 0); + base_type, 0, + explicit_p); if (count == 0) continue; @@ -1745,7 +1997,8 @@ expand_simd_clones (struct cgraph_node *node) /* And call the target hook again to get the right ISA. */ targetm.simd_clone.compute_vecsize_and_simdlen (node, clone, base_type, - i / 2); + i / 2, + explicit_p); if ((i & 1) != 0) clone->inbranch = 1; } @@ -1763,7 +2016,7 @@ expand_simd_clones (struct cgraph_node *node) /* Only when we are sure we want to create the clone actually clone the function (or definitions) or create another extern FUNCTION_DECL (for prototypes without definitions). */ - struct cgraph_node *n = simd_clone_create (node); + struct cgraph_node *n = simd_clone_create (node, !explicit_p); if (n == NULL) { if (i == 0) @@ -1798,6 +2051,10 @@ expand_simd_clones (struct cgraph_node *node) simd_clone_adjust_return_type (n); simd_clone_adjust_argument_types (n); } + if (dump_file) + fprintf (dump_file, "\nGenerated %s clone %s\n", + (TREE_PUBLIC (n->decl) ? "global" : "local"), + IDENTIFIER_POINTER (DECL_ASSEMBLER_NAME (n->decl))); } } while ((attr = lookup_attribute ("omp declare simd", TREE_CHAIN (attr)))); diff --git a/gcc/opts.cc b/gcc/opts.cc index 3797784c865..73fc97756e4 100644 --- a/gcc/opts.cc +++ b/gcc/opts.cc @@ -658,6 +658,8 @@ static const struct default_options default_options_table[] = REORDER_BLOCKS_ALGORITHM_STC }, { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_ftree_loop_vectorize, NULL, 1 }, { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_ftree_slp_vectorize, NULL, 1 }, + { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_fopenmp_target_simd_clone_, NULL, + OMP_TARGET_SIMD_CLONE_NOHOST }, #ifdef INSN_SCHEDULING /* Only run the pre-regalloc scheduling pass if optimizing for speed. */ { OPT_LEVELS_2_PLUS_SPEED_ONLY, OPT_fschedule_insns, NULL, 1 }, diff --git a/gcc/target.def b/gcc/target.def index d82606ff5ab..082a7c62f34 100644 --- a/gcc/target.def +++ b/gcc/target.def @@ -1634,7 +1634,7 @@ fields in @var{simd_clone} structure pointed by @var{clone_info} argument and al not determined by the bitsize (in which case @var{simdlen} is always used).\n\ The hook should return 0 if SIMD clones shouldn't be emitted,\n\ or number of @var{vecsize_mangle} variants that should be emitted.", -int, (struct cgraph_node *, struct cgraph_simd_clone *, tree, int), NULL) +int, (struct cgraph_node *, struct cgraph_simd_clone *, tree, int, bool), NULL) DEFHOOK (adjust, diff --git a/gcc/testsuite/g++.dg/gomp/target-simd-clone-1.C b/gcc/testsuite/g++.dg/gomp/target-simd-clone-1.C new file mode 100644 index 00000000000..10b5ac38812 --- /dev/null +++ b/gcc/testsuite/g++.dg/gomp/target-simd-clone-1.C @@ -0,0 +1,25 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are generated for functions with "declare target". */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* Although addit has external linkage, we expect clones to be generated as + for a function with internal linkage. */ + +/* { dg-final { scan-ipa-dump "Generated local clone _ZGV.*N.*__Z5additii" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump "Generated local clone _ZGV.*M.*__Z5additii" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/g++.dg/gomp/target-simd-clone-2.C b/gcc/testsuite/g++.dg/gomp/target-simd-clone-2.C new file mode 100644 index 00000000000..08c0539dc63 --- /dev/null +++ b/gcc/testsuite/g++.dg/gomp/target-simd-clone-2.C @@ -0,0 +1,23 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are not generated for "declare target" + functions that throw. */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + if (a < 0) throw -1; + return a + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-1.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-1.c new file mode 100644 index 00000000000..388dc2a956c --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-1.c @@ -0,0 +1,25 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are generated for functions with "declare target". */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* Although addit has external linkage, we expect clones to be generated as + for a function with internal linkage. */ + +/* { dg-final { scan-ipa-dump "Generated local clone _ZGV.*N.*_addit" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump "Generated local clone _ZGV.*M.*_addit" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-2.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-2.c new file mode 100644 index 00000000000..acdc241db46 --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-2.c @@ -0,0 +1,22 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fdump-ipa-simdclone-details" } */ + +/* Test that host simd clones are not generated for functions with + "declare target" by default at -O2. */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-3.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-3.c new file mode 100644 index 00000000000..e90d49ce7f2 --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-3.c @@ -0,0 +1,22 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that host simd clones are not generated for functions with the nohost + "declare target" clause. */ + +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} +#pragma omp declare target to(addit) device_type(nohost) + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* { dg-final { scan-ipa-dump "device doesn't match" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-4.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-4.c new file mode 100644 index 00000000000..141097bdce8 --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-4.c @@ -0,0 +1,26 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are not generated for functions with + "declare target" but that write memory in the body. */ + +extern int save; + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit(int a, int b) +{ + save = a; + return a + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* { dg-final { scan-ipa-dump "body includes memory write" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-5.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-5.c new file mode 100644 index 00000000000..93f9774f831 --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-5.c @@ -0,0 +1,28 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are not generated for functions with + "declare target" but unsuitable arguments. */ + +struct s { + int a; + int b; +}; + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (struct s x) +{ + return x.a + x.b; +} +#pragma omp end declare target + +void callit (struct s *ss, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (ss[i]); +} + +/* { dg-final { scan-ipa-dump "argument type fails sniff test" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-6.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-6.c new file mode 100644 index 00000000000..4c34967af95 --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-6.c @@ -0,0 +1,27 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are not generated for functions with + "declare target" but that call possibly side-effecting functions + in the body. */ + +extern int f (int); + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit(int a, int b) +{ + return f(a) + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* { dg-final { scan-ipa-dump "body contains unsafe function call" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ + diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-7.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-7.c new file mode 100644 index 00000000000..78c60f9f6bc --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-7.c @@ -0,0 +1,15 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are not generated for functions with + "declare target" that have no callers in the same compilation unit. */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} +#pragma omp end declare target + +/* { dg-final { scan-ipa-dump "function is not used" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/gcc.dg/gomp/target-simd-clone-8.c b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-8.c new file mode 100644 index 00000000000..561766cb128 --- /dev/null +++ b/gcc/testsuite/gcc.dg/gomp/target-simd-clone-8.c @@ -0,0 +1,25 @@ +/* { dg-options "-fopenmp -O2" } */ +/* { dg-additional-options "-fopenmp-target-simd-clone=any -fdump-ipa-simdclone-details" } */ + +/* Test that simd clones are not generated for functions with + "declare target" but that read volatile memory in the body. */ + +extern volatile int save; + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit(int a, int b) +{ + return save + a + b; +} +#pragma omp end declare target + +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} + +/* { dg-final { scan-ipa-dump "body includes volatile op" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ diff --git a/gcc/testsuite/lib/scanoffloadipa.exp b/gcc/testsuite/lib/scanoffloadipa.exp new file mode 100644 index 00000000000..615cbf46ef8 --- /dev/null +++ b/gcc/testsuite/lib/scanoffloadipa.exp @@ -0,0 +1,148 @@ +# Copyright (C) 2018-2022 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with GCC; see the file COPYING3. If not see +# . + +# Various utilities for scanning offloading ipa dump output, used by +# libgomp.exp. + +load_lib scandump.exp +load_lib scanoffload.exp + +# Utility for scanning compiler result, invoked via dg-final. +# Call pass if pattern is present, otherwise fail. +# +# Argument 0 is the regexp to match +# Argument 1 is the name of the dumped ipa pass +# Argument 2 handles expected failures and the like +proc scan-offload-ipa-dump { args } { + + if { [llength $args] < 2 } { + error "scan-offload-ipa-dump: too few arguments" + return + } + if { [llength $args] > 3 } { + error "scan-offload-ipa-dump: too many arguments" + return + } + if { [llength $args] >= 3 } { + scoff end-1 scan-dump "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" \ + [lindex $args 2] + } else { + scoff end scan-dump "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" + } +} + +# Call pass if pattern is present given number of times, otherwise fail. +# Argument 0 is the regexp to match +# Argument 1 is number of times the regexp must be found +# Argument 2 is the name of the dumped ipa pass +# Argument 3 handles expected failures and the like +proc scan-offload-ipa-dump-times { args } { + + if { [llength $args] < 3 } { + error "scan-offload-ipa-dump-times: too few arguments" + return + } + if { [llength $args] > 4 } { + error "scan-offload-ipa-dump-times: too many arguments" + return + } + if { [llength $args] >= 4 } { + scoff end-1 scan-dump-times "offload-ipa" [lindex $args 0] \ + [lindex $args 1] "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 2]" "" \ + [lindex $args 3] + } else { + scoff end scan-dump-times "offload-ipa" [lindex $args 0] \ + [lindex $args 1] "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 2]" "" + } +} + +# Call pass if pattern is not present, otherwise fail. +# +# Argument 0 is the regexp to match +# Argument 1 is the name of the dumped ipa pass +# Argument 2 handles expected failures and the like +proc scan-offload-ipa-dump-not { args } { + + if { [llength $args] < 2 } { + error "scan-offload-ipa-dump-not: too few arguments" + return + } + if { [llength $args] > 3 } { + error "scan-offload-ipa-dump-not: too many arguments" + return + } + if { [llength $args] >= 3 } { + scoff end-1 scan-dump-not "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" \ + [lindex $args 2] + } else { + scoff end scan-dump-not "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" + } +} + +# Utility for scanning demangled compiler result, invoked via dg-final. +# Call pass if pattern is present, otherwise fail. +# +# Argument 0 is the regexp to match +# Argument 1 is the name of the dumped ipa pass +# Argument 2 handles expected failures and the like +proc scan-offload-ipa-dump-dem { args } { + + if { [llength $args] < 2 } { + error "scan-offload-ipa-dump-dem: too few arguments" + return + } + if { [llength $args] > 3 } { + error "scan-offload-ipa-dump-dem: too many arguments" + return + } + if { [llength $args] >= 3 } { + scoff end-1 scan-dump-dem "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" \ + [lindex $args 2] + } else { + scoff end scan-dump-dem "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" + } +} + +# Call pass if demangled pattern is not present, otherwise fail. +# +# Argument 0 is the regexp to match +# Argument 1 is the name of the dumped ipa pass +# Argument 2 handles expected failures and the like +proc scan-offload-ipa-dump-dem-not { args } { + + if { [llength $args] < 2 } { + error "scan-offload-ipa-dump-dem-not: too few arguments" + return + } + if { [llength $args] > 3 } { + error "scan-offload-ipa-dump-dem-not: too many arguments" + return + } + if { [llength $args] >= 3 } { + scoff end-1 scan-dump-dem-not "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" \ + [lindex $args 2] + } else { + scoff end scan-dump-dem-not "offload-ipa" [lindex $args 0] \ + "\[0-9\]\[0-9\]\[0-9]i.[lindex $args 1]" "" + } +} diff --git a/libgomp/testsuite/lib/libgomp.exp b/libgomp/testsuite/lib/libgomp.exp index 1801fdc8224..e12236e9083 100644 --- a/libgomp/testsuite/lib/libgomp.exp +++ b/libgomp/testsuite/lib/libgomp.exp @@ -32,6 +32,7 @@ load_gcc_lib scansarif.exp load_gcc_lib scantree.exp load_gcc_lib scanltranstree.exp load_gcc_lib scanoffload.exp +load_gcc_lib scanoffloadipa.exp load_gcc_lib scanoffloadtree.exp load_gcc_lib scanoffloadrtl.exp load_gcc_lib scanipa.exp diff --git a/libgomp/testsuite/libgomp.c/target-simd-clone-1.c b/libgomp/testsuite/libgomp.c/target-simd-clone-1.c new file mode 100644 index 00000000000..a9defc4cdd6 --- /dev/null +++ b/libgomp/testsuite/libgomp.c/target-simd-clone-1.c @@ -0,0 +1,43 @@ +/* { dg-do link { target { offload_target_amdgcn } } } */ +/* { dg-additional-options "-O2 -foffload-options=-fdump-ipa-simdclone-details" } */ + +/* Test that simd clones for the offload processor are generated for + functions with "declare target" when enabled by default at -O2. */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} + +__attribute__ ((__noinline__)) +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} +#pragma omp end declare target + +int main (void) +{ + int aa[16], bb[16], cc[16]; + int i; + for (i = 0; i < 16; i++) + { + aa[i] = i; + bb[i] = -i; + } + callit (aa, bb, cc); + for (i = 0; i < 16; i++) + if (cc[i] != 0) + return 1; + return 0; +} + +/* Although addit has external linkage, we expect clones to be generated as + for a function with internal linkage. */ + +/* { dg-final { scan-offload-ipa-dump "Generated local clone _ZGV.*N.*_addit" "simdclone" } } */ +/* { dg-final { scan-offload-ipa-dump "Generated local clone _ZGV.*M.*_addit" "simdclone" } } */ diff --git a/libgomp/testsuite/libgomp.c/target-simd-clone-2.c b/libgomp/testsuite/libgomp.c/target-simd-clone-2.c new file mode 100644 index 00000000000..05a38ae2bc5 --- /dev/null +++ b/libgomp/testsuite/libgomp.c/target-simd-clone-2.c @@ -0,0 +1,39 @@ +/* { dg-do link { target { offload_target_amdgcn } } } */ +/* { dg-additional-options "-foffload-options=-fdump-ipa-simdclone-details -foffload-options=-fno-openmp-target-simd-clone" } */ + +/* Test that simd clones for the offload processor are not generated for + functions with "declare target" when explicitly disabled. */ + +#pragma omp declare target +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} + +__attribute__ ((__noinline__)) +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} +#pragma omp end declare target + +int main (void) +{ + int aa[16], bb[16], cc[16]; + int i; + for (i = 0; i < 16; i++) + { + aa[i] = i; + bb[i] = -i; + } + callit (aa, bb, cc); + for (i = 0; i < 16; i++) + if (cc[i] != 0) + return 1; + return 0; +} + +/* { dg-final { scan-offload-ipa-dump-not "Generated .* clone" "simdclone" } } */ diff --git a/libgomp/testsuite/libgomp.c/target-simd-clone-3.c b/libgomp/testsuite/libgomp.c/target-simd-clone-3.c new file mode 100644 index 00000000000..bde091e24ba --- /dev/null +++ b/libgomp/testsuite/libgomp.c/target-simd-clone-3.c @@ -0,0 +1,40 @@ +/* { dg-do link { target { offload_target_amdgcn } } } */ +/* { dg-additional-options "-O2 -foffload-options=-fdump-ipa-simdclone-details" } */ + +/* Test that device simd clones are not generated for functions with the host + "declare target" clause only. */ + +__attribute__ ((__noinline__)) int addit (int a, int b) +{ + return a + b; +} +#pragma omp declare target to(addit) device_type(host) + +#pragma omp declare target +void callit (int *a, int *b, int *c) +{ + int i; + #pragma omp for simd + for (i = 0; i < 16; i++) + c[i] = addit (a[i], b[i]); +} +#pragma omp end declare target + +int main (void) +{ + int aa[16], bb[16], cc[16]; + int i; + for (i = 0; i < 16; i++) + { + aa[i] = i; + bb[i] = -i; + } + callit (aa, bb, cc); + for (i = 0; i < 16; i++) + if (cc[i] != 0) + return 1; + return 0; +} + +/* { dg-final { scan-offload-ipa-dump "device doesn't match" "simdclone" { target x86_64-*-* } } } */ +/* { dg-final { scan-offload-ipa-dump-not "Generated .* clone" "simdclone" { target x86_64-*-* } } } */ -- 2.31.1