From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 108441 invoked by alias); 17 Oct 2016 17:08:05 -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 108419 invoked by uid 89); 17 Oct 2016 17:08:04 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.8 required=5.0 tests=AWL,BAYES_00,RCVD_IN_BRBL_LASTEXT,RP_MATCHES_RCVD,SPF_PASS autolearn=no version=3.3.2 spammy=Hx-languages-length:1410, UD:php X-HELO: smtp.ispras.ru Received: from bran.ispras.ru (HELO smtp.ispras.ru) (83.149.199.196) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 17 Oct 2016 17:07:53 +0000 Received: from monopod.intra.ispras.ru (monopod.intra.ispras.ru [10.10.3.121]) by smtp.ispras.ru (Postfix) with ESMTP id CAD6D61284; Mon, 17 Oct 2016 20:07:50 +0300 (MSK) Date: Mon, 17 Oct 2016 17:08:00 -0000 From: Alexander Monakov To: Bernd Schmidt cc: gcc-patches@gcc.gnu.org, Nathan Sidwell Subject: Re: [PATCH 0/8] NVPTX offloading to NVPTX: backend patches In-Reply-To: <6e6106d4-3ae9-54e2-c1d3-99a2580297dc@redhat.com> Message-ID: References: <1476463189-22540-1-git-send-email-amonakov@ispras.ru> <6e6106d4-3ae9-54e2-c1d3-99a2580297dc@redhat.com> User-Agent: Alpine 2.20.13 (LNX 116 2015-12-14) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-SW-Source: 2016-10/txt/msg01331.txt.bz2 On Mon, 17 Oct 2016, Bernd Schmidt wrote: > On 10/14/2016 06:39 PM, Alexander Monakov wrote: > > I'm resending the patch series with backend prerequisites for OpenMP > > offloading to the NVIDIA PTX ISA. The patches are rebased on trunk. > > What's the status of the branch? Is it expected to work? I'm trying to compile > the OpenMP version of these benchmarks: > https://codesign.llnl.gov/lulesh.php > > and the resulting binary fails as follows: > > libgomp: Link error log error : Size doesn't match for '__nvptx_stacks' in > 'Input 8', first specified in 'Input 8' > error : Multiple definition of '__nvptx_stacks' in 'Input 8', first defined > in 'Input 8' I've just pushed two commits to the branch to fix this issue. Before those, the last commit left the branch in a state where an incremental build seemed ok (because libgcc/libgomp weren't rebuilt with the new cc1), but a from-scratch build was broken like you've shown. LULESH is known to work. I also intend to perform a trunk merge soon. > I think before merging this work we'll need to have some idea of how well it > works on real-world code. This patchset and the branch lay the foundation, there's more work to be done, in particular on the performance improvements side. There should be an agreement on these fundamental bits first, before moving on to fine-tuning. Alexander