From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28186 invoked by alias); 25 Jan 2011 07:03:13 -0000 Received: (qmail 28178 invoked by uid 22791); 25 Jan 2011 07:03:12 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 25 Jan 2011 07:03:06 +0000 Received: (qmail 32398 invoked from network); 25 Jan 2011 07:03:03 -0000 Received: from unknown (HELO ?192.168.1.16?) (cltang@127.0.0.2) by mail.codesourcery.com with ESMTPA; 25 Jan 2011 07:03:03 -0000 Message-ID: <4D3E75AB.3010406@codesourcery.com> Date: Tue, 25 Jan 2011 09:43:00 -0000 From: Chung-Lin Tang User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.13) Gecko/20101207 Thunderbird/3.1.7 MIME-Version: 1.0 To: gcc-patches CC: ramana.radhakrishnan@arm.com, Richard Earnshaw Subject: [patch, ARM] PR47246, VFP index range on Thumb-2 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit 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-01/txt/msg01719.txt.bz2 Hi, This fixes a condition I overlooked in a prior patch. The allowed Thumb-2 load/store index range for core registers is -256--4096 exclusive, while the coprocessor range is still -1024--1024 like ARM mode. The valid intersection is then -256--1024. This actually relates to the problem of having different valid memory addresses for the same machine mode when load/storing to different register classes (here SF/DF modes stored in both GPR and FPU regs, with differing index ranges), which does not seem to be an easy problem within GCC. So here's the conservative patch, again tested without regressions. It corrects other coprocessor cases (FPA, Maverick) together, which should be correct (not that I've seen Thumb-2 implementations with them) Okay for trunk? Thanks, Chung-Lin