From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 74760 invoked by alias); 17 Jul 2017 12:33:32 -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 74084 invoked by uid 89); 17 Jul 2017 12:33:31 -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=H*M:internal, Hx-languages-length:407, H*RU:14.03.0266.001, H*r:14.03.0266.001 X-HELO: smtprelay.synopsys.com Received: from smtprelay.synopsys.com (HELO smtprelay.synopsys.com) (198.182.47.9) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 17 Jul 2017 12:33:30 +0000 Received: from mailhost.synopsys.com (mailhost2.synopsys.com [10.13.184.66]) by smtprelay.synopsys.com (Postfix) with ESMTP id 2C6CD24E1594; Mon, 17 Jul 2017 05:33:27 -0700 (PDT) Received: from mailhost.synopsys.com (localhost [127.0.0.1]) by mailhost.synopsys.com (Postfix) with ESMTP id C0F226EF; Mon, 17 Jul 2017 05:33:27 -0700 (PDT) Received: from US01WEHTC2.internal.synopsys.com (us01wehtc2.internal.synopsys.com [10.12.239.237]) by mailhost.synopsys.com (Postfix) with ESMTP id 8DB356E9; Mon, 17 Jul 2017 05:33:27 -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; Mon, 17 Jul 2017 05:32:52 -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; Mon, 17 Jul 2017 14:32:50 +0200 From: Claudiu Zissulescu To: Andrew Burgess CC: "gcc-patches@gcc.gnu.org" , "Francois.Bedard@synopsys.com" Subject: RE: [PATCH 5/7] [ARC] Enable indexed loads for elf targers. Date: Mon, 17 Jul 2017 12:33:00 -0000 Message-ID: <098ECE41A0A6114BB2A07F1EC238DE8966630583@de02wembxa.internal.synopsys.com> References: <1496324097-21221-1-git-send-email-claziss@synopsys.com> <1496324097-21221-6-git-send-email-claziss@synopsys.com> <20170713115553.GE27353@embecosm.com> In-Reply-To: <20170713115553.GE27353@embecosm.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-SW-Source: 2017-07/txt/msg00971.txt.bz2 > The change looks fine, but it would be nice if the commit message > explained _why_ we are default off for Linux and on for Elf, I think > more text in the commit message on this sort of thing will help future > developers understand why things are the way they are. >=20 Committed with suggested annotations. Thank you for your review, Claudiu