From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9413 invoked by alias); 8 Oct 2015 10:12:39 -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 9400 invoked by uid 89); 8 Oct 2015 10:12:38 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.2 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-vk0-f51.google.com Received: from mail-vk0-f51.google.com (HELO mail-vk0-f51.google.com) (209.85.213.51) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Thu, 08 Oct 2015 10:12:37 +0000 Received: by vkgd64 with SMTP id d64so28867961vkg.0 for ; Thu, 08 Oct 2015 03:12:35 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.31.171.88 with SMTP id u85mr4350420vke.7.1444299155216; Thu, 08 Oct 2015 03:12:35 -0700 (PDT) Received: by 10.103.65.1 with HTTP; Thu, 8 Oct 2015 03:12:35 -0700 (PDT) In-Reply-To: <56163D75.4080805@redhat.com> References: <1444280375-20866-1-git-send-email-rth@redhat.com> <1444280375-20866-10-git-send-email-rth@redhat.com> <56163D75.4080805@redhat.com> Date: Thu, 08 Oct 2015 10:12:00 -0000 Message-ID: Subject: Re: [PATCH 9/9] Fix PR 66768 From: "Bin.Cheng" To: Bernd Schmidt Cc: Richard Henderson , gcc-patches List Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-10/txt/msg00813.txt.bz2 On Thu, Oct 8, 2015 at 5:55 PM, Bernd Schmidt wrote: > On 10/08/2015 07:17 AM, Bin.Cheng wrote: >> >> On Thu, Oct 8, 2015 at 12:59 PM, Richard Henderson wrote: >>> >>> This is the patch that richi includes in the PR. There will need to >>> be an additional patch to solve an ICE for the AVR backend, as noted >>> in the PR, but this is good enough to solve the bad-code generation >>> problem for the i386 backend. >> >> Hi Richard, >> For the record, the root cause is in IVO because it fails to preserve >> base object. This patch can only paper over the issue for address >> spaces where PTR type and sizetype have the same length, otherwise IVO >> generates wrong code which can't be walked around by this patch. I >> will take PR66768. > > > Hmm. In 2012 I submitted a patch "Preserve pointer types in ivopts", which > got lost in review. It was for a different problem than address spaces, but > it might be worth taking a look whether that approach could help solve this > issue. Hi Bernd, Thanks for your suggestion, I will search for that patch. Thanks, bin > > > Bernd