From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23953 invoked by alias); 22 Nov 2011 10:27:03 -0000 Received: (qmail 23944 invoked by uid 22791); 22 Nov 2011 10:27:02 -0000 X-SWARE-Spam-Status: No, hits=-0.5 required=5.0 tests=AWL,BAYES_40,RCVD_IN_DNSWL_LOW,TW_IW,TW_MX X-Spam-Check-By: sourceware.org Received: from service87.mimecast.com (HELO service87.mimecast.com) (91.220.42.44) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 22 Nov 2011 10:26:37 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Tue, 22 Nov 2011 10:26:32 +0000 Received: from [10.1.79.40] ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.0); Tue, 22 Nov 2011 10:26:28 +0000 Subject: RE: [RFA/ARM][Patch 01/02]: Thumb2 epilogue in RTL From: Sameera Deshpande To: Xinyu Qi Cc: Ramana Radhakrishnan , "gcc-patches@gcc.gnu.org" In-Reply-To: <4737A960563B524DA805CA602BE04B3060291A4EDB@SC-VEXCH2.marvell.com> References: <4EBBD52A.6040707@arm.com> <1320948467.17411.17.camel@e102549-lin.cambridge.arm.com> <4737A960563B524DA805CA602BE04B3060291A4EDB@SC-VEXCH2.marvell.com> Date: Tue, 22 Nov 2011 12:16:00 -0000 Message-ID: <1321957587.15343.6.camel@e102549-lin.cambridge.arm.com> Mime-Version: 1.0 X-MC-Unique: 111112210263211001 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes 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 X-SW-Source: 2011-11/txt/msg02205.txt.bz2 On Tue, 2011-11-22 at 01:55 +0000, Xinyu Qi wrote: > At 2011-11-19 07:11:17,"Ramana Radhakrishnan" wrote: > > On 10 November 2011 18:07, Sameera Deshpande > > wrote: > > > Please find attached the reworked patch. > >=20 > > OK but for a very small bit . > >=20 > > I'll note that we need to add support for the iwMMXt registers but the > > attached patch (untested) should be what is needed to support the iwMMxt > > registers and this should be something that should be incorporated into= your > > ARM RTL epilogues patch as well. My understanding is that this is all y= ou > > should need to do as there is a wldrd as a part of *iwmmx*_movdi instru= ction > > and therefore this should just work in this form. > >=20 > > Xinyu can you help in testing this patch once this is applied along wit= h your > > other patches to turn on Thumb2 for iwmmxt ? > >=20 > > cheers > > Ramana >=20 > Hi, Ramana, >=20 > Though I don't fully understand this patch, I think it is unnecessary to = consider turn on Thumb2 for iwmmxt, > because there is no chip that supports WMMX instructions with thumb2 mode. >=20 > Thanks, > Xinyu >=20 Ramana, in that case, should I add the change you suggested in ARM RTL epilogue patch only? --=20