From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4459 invoked by alias); 13 Jul 2017 13:08:33 -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 3969 invoked by uid 89); 13 Jul 2017 13:08:33 -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:08:31 +0000 Received: from mailhost.synopsys.com (mailhost3.synopsys.com [10.12.238.238]) by smtprelay.synopsys.com (Postfix) with ESMTP id 2B29A24E1F2A; Thu, 13 Jul 2017 06:08:30 -0700 (PDT) Received: from mailhost.synopsys.com (localhost [127.0.0.1]) by mailhost.synopsys.com (Postfix) with ESMTP id 1769F5CD; Thu, 13 Jul 2017 06:08:30 -0700 (PDT) Received: from US01WXQAHTC1.internal.synopsys.com (us01wxqahtc1.internal.synopsys.com [10.12.238.230]) by mailhost.synopsys.com (Postfix) with ESMTP id E224F5C8; Thu, 13 Jul 2017 06:08:29 -0700 (PDT) Received: from DE02WEHTCA.internal.synopsys.com (10.225.19.92) by US01WXQAHTC1.internal.synopsys.com (10.12.238.230) with Microsoft SMTP Server (TLS) id 14.3.266.1; Thu, 13 Jul 2017 06:08:29 -0700 Received: from DE02WEMBXA.internal.synopsys.com ([fe80::a014:7216:77d:d55c]) by DE02WEHTCA.internal.synopsys.com ([::1]) with mapi id 14.03.0266.001; Thu, 13 Jul 2017 15:08:27 +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: Thu, 13 Jul 2017 13:08:00 -0000 Message-ID: <098ECE41A0A6114BB2A07F1EC238DE896663016F@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/msg00734.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 This explanation is quite simple, we haven't fully validated Linux with th= ose options on. As it may take a while until I am getting a more complete i= mage on this enhancement for Linux, I've enabled this option for elf, where= I've got very good feedback from our gnu users. Normally, it should be no problem to enable them for Linux as well, but I w= ould like to have more testing data on this subject from various gnu users. Thanks, Claudiu