From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8847 invoked by alias); 11 Aug 2017 15:38:52 -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 8614 invoked by uid 89); 11 Aug 2017 15:38:50 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.7 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,SPF_PASS,URIBL_RED autolearn=ham version=3.3.2 spammy= X-HELO: relay1.mentorg.com Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 11 Aug 2017 15:38:47 +0000 Received: from svr-orw-fem-03.mgc.mentorg.com ([147.34.97.39]) by relay1.mentorg.com with esmtp id 1dgC1N-0002L6-Gp from Thomas_Schwinge@mentor.com for gcc-patches@gcc.gnu.org; Fri, 11 Aug 2017 08:38:45 -0700 Received: from tftp-cs (147.34.91.1) by svr-orw-fem-03.mgc.mentorg.com (147.34.97.39) with Microsoft SMTP Server id 14.3.224.2; Fri, 11 Aug 2017 08:38:40 -0700 Received: by tftp-cs (Postfix, from userid 49978) id C21A5C2301; Fri, 11 Aug 2017 08:38:39 -0700 (PDT) From: Thomas Schwinge To: Tom de Vries , GCC Patches Subject: Re: [committed, nvptx] Add nvptx_override_options_after_change In-Reply-To: <6661dd7d-f22e-b043-5b74-792b3c74f08f@mentor.com> References: <767255ef-911c-d89c-3569-785c30354166@mentor.com> <65b84334-0d77-2507-fb65-f94c87ce8d1c@mentor.com> <87shh230t1.fsf@euler.schwinge.homeip.net> <6661dd7d-f22e-b043-5b74-792b3c74f08f@mentor.com> User-Agent: Notmuch/0.9-125-g4686d11 (http://notmuchmail.org) Emacs/25.2.1 (x86_64-pc-linux-gnu) Date: Fri, 11 Aug 2017 16:46:00 -0000 Message-ID: <87k22a2jyg.fsf@euler.schwinge.homeip.net> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-SW-Source: 2017-08/txt/msg00798.txt.bz2 Hi! On Tue, 8 Aug 2017 10:58:12 +0200, Tom de Vries wr= ote: > On 08/08/2017 10:45 AM, Thomas Schwinge wrote: > > On Fri, 21 Jul 2017 11:49:11 +0200, Tom de Vries wrote: > >> this patch adds nvptx_override_options_after_change, containing a > >> workaround for PR81430. > > Should this be reverted now that r250852 "Apply finish_options on > > DECL_FUNCTION_SPECIFIC_OPTIMIZATION for ACCEL_COMPILER" has been > > committed? (I'm confirming this works fine, thanks.) >=20 > Indeed, it can be reverted. Committed to trunk in r251053: commit 5b005e86c8c45aa84c090064b2f789c802384fc5 Author: tschwinge Date: Fri Aug 11 15:37:14 2017 +0000 [PR lto/81430] Revert "Add nvptx_override_options_after_change" =20=20=20=20 This reverts r250421; properly fixed by r250852. =20=20=20=20 PR lto/81430 * config/nvptx/nvptx.c (nvptx_override_options_after_change): Remove function. (TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE): Remove definition. =20=20=20=20 git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@251053 138bc75d-0d04-04= 10-961f-82ee72b054a4 --- gcc/ChangeLog | 7 +++++++ gcc/config/nvptx/nvptx.c | 14 -------------- 2 files changed, 7 insertions(+), 14 deletions(-) diff --git gcc/ChangeLog gcc/ChangeLog index 2975655..295b4c9 100644 --- gcc/ChangeLog +++ gcc/ChangeLog @@ -1,3 +1,10 @@ +2017-08-11 Thomas Schwinge + + PR lto/81430 + * config/nvptx/nvptx.c (nvptx_override_options_after_change): + Remove function. + (TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE): Remove definition. + 2017-08-11 Tamar Christina * config/aarch64/aarch64.md (mov): Change. (*movhf_aarch64, *movsf_aarch64, *movdf_aarch64): diff --git gcc/config/nvptx/nvptx.c gcc/config/nvptx/nvptx.c index ffd50d7..8babac7 100644 --- gcc/config/nvptx/nvptx.c +++ gcc/config/nvptx/nvptx.c @@ -212,17 +212,6 @@ nvptx_option_override (void) target_flags |=3D MASK_SOFT_STACK | MASK_UNIFORM_SIMT; } =20 -/* Implement TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE. */ - -static void -nvptx_override_options_after_change (void) -{ - /* This is a workaround for PR81430 - nvptx acceleration compilation bro= ken - because of running pass_partition_blocks. This should be dealt with = in the - common code, not in the target. */ - flag_reorder_blocks_and_partition =3D 0; -} - /* Return a ptx type for MODE. If PROMOTE, then use .u32 for QImode to deal with ptx ideosyncracies. */ =20 @@ -5527,9 +5516,6 @@ nvptx_data_alignment (const_tree type, unsigned int b= asic_align) #undef TARGET_OPTION_OVERRIDE #define TARGET_OPTION_OVERRIDE nvptx_option_override =20 -#undef TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE -#define TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE nvptx_override_options_after_= change - #undef TARGET_ATTRIBUTE_TABLE #define TARGET_ATTRIBUTE_TABLE nvptx_attribute_table =20 Gr=C3=BC=C3=9Fe Thomas