From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30458 invoked by alias); 9 Mar 2005 09:24:37 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Received: (qmail 29348 invoked by alias); 9 Mar 2005 09:23:56 -0000 Date: Wed, 09 Mar 2005 09:24:00 -0000 Message-ID: <20050309092356.29347.qmail@sourceware.org> From: "jakub at redhat dot com" To: gcc-bugs@gcc.gnu.org In-Reply-To: <20050221214433.20126.jkohen@users.sourceforge.net> References: <20050221214433.20126.jkohen@users.sourceforge.net> Reply-To: gcc-bugzilla@gcc.gnu.org Subject: [Bug target/20126] [3.3/3.4/4.0/4.1 Regression] Inlined memcmp makes one argument null on entry X-Bugzilla-Reason: CC X-SW-Source: 2005-03/txt/msg01141.txt.bz2 List-Id: ------- Additional Comments From jakub at redhat dot com 2005-03-09 09:23 ------- Subject: Re: [PR target/20126, RFC] loop DEST_ADDR biv replacement may fail On Wed, Mar 09, 2005 at 01:02:08AM -0300, Alexandre Oliva wrote: > On Mar 8, 2005, Jakub Jelinek wrote: > > > Unfortunately, it seems to break ada bootstrap on at least x86-64 and i386: > > Odd... It surely completed bootstrap for me with default build flags. Your code hits just once on ali.adb stage1/gnat1 -I- -I. -Iada -I../../gcc/ada -quiet -dumpbase ali.adb -O2 -gnatpg \ -gnata -gnatO ada/ali.o ../../gcc/ada/ali.adb -o foo.s (gdb) p debug_rtx(v->insn) (insn 1571 1568 1573 (set (mem/s/v/j:QI (plus:SI (reg:SI 349 [ D.4388 ]) (symbol_ref:SI ("ali__cumulative_restrictions") [flags 0x2] )) [5 ali__cumulative_restrictions.set S1 A8]) (const_int 1 [0x1])) -1 (nil) (nil)) (gdb) p debug_rtx(*v->location) (plus:SI (reg:SI 349 [ D.4388 ]) (symbol_ref:SI ("ali__cumulative_restrictions") [flags 0x2] )) (gdb) p debug_rtx(v->new_reg) (plus:SI (reg:SI 1953) (symbol_ref:SI ("ali__cumulative_restrictions") [flags 0x2] )) Note that recog_memoized (v->insn) is -1 even without the change, not sure what would fix that up. But it certainly shows that *v->location doesn't have to be a simple pseudo, so gen_move_insn might not work. Jakub -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20126