From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 108650 invoked by alias); 13 Jul 2017 13:31:48 -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 108638 invoked by uid 89); 13 Jul 2017 13:31:47 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD autolearn=no version=3.3.2 spammy=Hx-spam-relays-external:sk:mailhos, H*RU:sk:mailhos, our X-HELO: smtprelay.synopsys.com Received: from smtprelay4.synopsys.com (HELO smtprelay.synopsys.com) (198.182.47.9) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 13 Jul 2017 13:31:46 +0000 Received: from mailhost.synopsys.com (mailhost3.synopsys.com [10.12.238.238]) by smtprelay.synopsys.com (Postfix) with ESMTP id 2558B24E1F2A; Thu, 13 Jul 2017 06:31:45 -0700 (PDT) Received: from mailhost.synopsys.com (localhost [127.0.0.1]) by mailhost.synopsys.com (Postfix) with ESMTP id 08043826; Thu, 13 Jul 2017 06:31:45 -0700 (PDT) Received: from US01WEHTC2.internal.synopsys.com (us01wehtc2-vip.internal.synopsys.com [10.12.239.238]) by mailhost.synopsys.com (Postfix) with ESMTP id 2C36680D; Thu, 13 Jul 2017 06:31:44 -0700 (PDT) Received: from DE02WEHTCB.internal.synopsys.com (10.225.19.94) by US01WEHTC2.internal.synopsys.com (10.12.239.237) with Microsoft SMTP Server (TLS) id 14.3.266.1; Thu, 13 Jul 2017 06:31:44 -0700 Received: from DE02WEMBXA.internal.synopsys.com ([fe80::a014:7216:77d:d55c]) by DE02WEHTCB.internal.synopsys.com ([::1]) with mapi id 14.03.0266.001; Thu, 13 Jul 2017 15:31:42 +0200 From: Claudiu Zissulescu To: Andrew Burgess CC: "gcc-patches@gcc.gnu.org" , "Francois.Bedard@synopsys.com" Subject: RE: [PATCH 6/7] [ARC] Deprecate mexpand-adddi option. Date: Thu, 13 Jul 2017 13:31:00 -0000 Message-ID: <098ECE41A0A6114BB2A07F1EC238DE8966630195@de02wembxa.internal.synopsys.com> References: <1496324097-21221-1-git-send-email-claziss@synopsys.com> <1496324097-21221-7-git-send-email-claziss@synopsys.com> <20170713131803.GF27353@embecosm.com> In-Reply-To: <20170713131803.GF27353@embecosm.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-SW-Source: 2017-07/txt/msg00738.txt.bz2 >=20 > This looks fine, though the commit message tells me it's not a good > idea, but it would be nice to know _why_ it's not good. Might be nice > to know for future reference. >=20 Again LRA, expand time subregs are not handled by LRA, as far as I can tell= . Moreover, this option is by default off, and even using the old IRA may n= ot be a good idea to have subregs before register allocation step. > Also, there's no test. Was there an issue that revealed this as not a > good idea? Could that become a test? >=20 This error was found while running dg for our port with LRA on and this opt= ion on. Once removed the mexpand-adddi, a test is impossible to get. Thanks, Claudiu