From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28769 invoked by alias); 23 Jan 2017 13:11:50 -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 28751 invoked by uid 89); 23 Jan 2017 13:11:49 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.3 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=no version=3.3.2 spammy=his, morning X-HELO: mail-oi0-f66.google.com Received: from mail-oi0-f66.google.com (HELO mail-oi0-f66.google.com) (209.85.218.66) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 23 Jan 2017 13:11:39 +0000 Received: by mail-oi0-f66.google.com with SMTP id u143so10627135oif.3 for ; Mon, 23 Jan 2017 05:11:39 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:content-transfer-encoding; bh=2VAxnZHXHX067Wst9L9/DNaYsHRPPH3g/MuI7v+S22Q=; b=MFIuXHUjixVGkoYgmCj3wA/2A23WTQEj+kHJCgvxjtHr/Q90yT85WM76LPa9EDHgYY pLuf8yDVmywavOT8JILwXRVItWA6Z/eP6HVn3hu2tYqK8N7mqEiA96nP1lTcjyZ7Obbl 9/E/uBsk+9argNuG8f8xT6G6eKOjo6hd7vk4pSj7oyhBblnru9w0hLlYNmQVtBo8GGWN RF8xpKtsvj4pJzEvTnXz10cZ/wlkIgyisY+S9+KCwbunXxd3BUDFFttkrsMM5tU/OVWy RmGkfgJ41Ofltuq2iZCu8ioy6FXgC5IEipvVDh+WECthL3kp0008yuam8vbqRyKVnfQ8 2pTA== X-Gm-Message-State: AIkVDXJXePpsYhRNmcBaB2SijZTel8IGSt9PJXycpHI+7MNqLPQisppwrYouMBZlWV8rwIXlRZDcy+p5PDNVhw== X-Received: by 10.202.78.195 with SMTP id c186mr13042404oib.164.1485177097952; Mon, 23 Jan 2017 05:11:37 -0800 (PST) MIME-Version: 1.0 Received: by 10.157.42.12 with HTTP; Mon, 23 Jan 2017 05:11:37 -0800 (PST) In-Reply-To: <20170123120240.h6kannl7ygryurbj@virgil.suse.cz> References: <20170116090746.GM1867@tucnak> <20170123120240.h6kannl7ygryurbj@virgil.suse.cz> From: Richard Biener Date: Mon, 23 Jan 2017 13:17:00 -0000 Message-ID: Subject: Re: [PATCH] BRIG frontend: request for a global review To: Richard Biener , =?UTF-8?B?UGVra2EgSsOkw6Rza2Vsw6RpbmVu?= , Jakub Jelinek , GCC Patches Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2017-01/txt/msg01762.txt.bz2 On Mon, Jan 23, 2017 at 1:02 PM, Martin Jambor wrote: > Hi, > > > On Mon, Jan 23, 2017 at 12:56:13PM +0100, Richard Biener wrote: >> On Fri, Jan 20, 2017 at 6:25 PM, Pekka J=C3=A4=C3=A4skel=C3=A4inen wrote: >> > Hi Richard, >> > >> > On Fri, Jan 20, 2017 at 10:26 AM, Richard Biener >> > wrote: >> >> So the #ifdef ENABLE_BRIG_FE shouldn't be needed anymore (nor the >> >> configury for it). >> >> >> >> Otherwise this looks ok to me then. >> > >> > Attached is a patch set with that unnecessary ENABLE_BRIG_FE macro >> > removed. It has also been refreshed to the latest trunk. >> >> Are you set up to commit these to trunk yourself or do you need help her= e? >> > > Pekka is still in the process of requesting an account at gcc.gnu.org. > We agreed that I would commit the patches. > > Over Saturday and today morning, I have bootstrappet and tested them > in various configurations (although mostly on x86_64 and only a little > bit on aarch64) and (together with a "svn mv libgomp/plugin/hsa.h > include/") I have not found any issue so far. > > I suppose I should go ahead (in any case, if there is any fallout, > reach out to me too, at least until Pekka gets his account). Yes, sounds good to me. Richard. > Thanks, > > Martin >