[Bug target/15286] ICE cause by reload

--- Additional Comments From pinskia at gcc dot gnu dot org 2004-10-28 13:44 --- Fixed. -- What|Removed |Added Status|NEW |RESOLVED

[Bug target/15286] ICE cause by reload

--- Additional Comments From cvs-commit at gcc dot gnu dot org 2004-10-28 12:47 --- Subject: Bug 15286 CVSROOT:/cvs/gcc Module name:gcc Changes by: [EMAIL PROTECTED] 2004-10-28 12:47:29 Modified files: gcc: ChangeLog final.c recog.c simplify-rt

[Bug target/15286] ICE cause by reload

--- Additional Comments From pinskia at gcc dot gnu dot org 2004-10-27 15:56 --- Patch here: . -- What|Removed |Added

[Bug target/15286] ICE cause by reload

--- Additional Comments From fjahanian at apple dot com 2004-10-26 15:17 --- I tested the patch on apple-ppc-darwin; bootstrapped and dejagnu tested (with and without -mcpu=G5). There were no regressions. This is an important bug for us. We have had 4 separate reporting of this bug. It

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-26 04:28 --- Subject: Re: ICE cause by reload I found a typo in my sourcebase introduced when applying one of the patches. Fixing that typo (matching the complete patch attached to the PR) produces the correct

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-26 00:13 --- Subject: Re: ICE cause by reload There is a reduced testcase attached to the Bugzilla PR. Please do not confuse discussion with other examples. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id

[Bug target/15286] ICE cause by reload

--- Additional Comments From fjahanian at apple dot com 2004-10-25 23:58 --- I tried the last patch and for the following statement built with -O2 -mcpu=G5 (aaple's mixed mode) I get the following instruction sequence. It looks OK to me. But David's case might be different than what I a

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-25 23:43 --- Subject: Re: ICE cause by reload Try again with what patch? I have all patches applied and I consistently get the output I emailed earlier with a native Mac OS X compiler. David -- http://g

[Bug target/15286] ICE cause by reload

--- Additional Comments From fjahanian at apple dot com 2004-10-25 21:14 --- By mistake, I applied the test for !reload_completed to you earlier patch (which was worng). In any case, after correcting the patch and with your latest patch, all my test cases passed. Now, I need to do a c

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-25 21:11 --- Subject: Re: ICE cause by reload With the earlier *two* patches from Ulrich, the compiler no longer ICE. The latest patch to alter_subreg() (with the missing GET_MODEs) does not fix the s

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-25 21:04 --- well, as pointed out by David you do need in addition the change I described in comment #20. If you prefer a patch: Index: gcc/recog.c ===

[Bug target/15286] ICE cause by reload

--- Additional Comments From fjahanian at apple dot com 2004-10-25 20:58 --- You need to replace GET_MODE_SIZE (x) with GET_MODE_SIZE (GET_MODE (x)), etc. for a clean compile. But as I mentioned in last comment, I still get the ICE with or without this patch (along with the previous pat

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-25 20:42 --- (My last message got messed up somehow, I'm trying again.) David Edelsohn wrote: >Correction. While the reload changes fix the crash, it looks like there still >is a bug because the resulting code do

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-25 20:38 --- David Edelsohn wrote: >Correction. While the reload changes fix the crash, it looks like there still >is a bug because the resulting code does not access the correct SUBREG. It >does not access the L

[Bug target/15286] ICE cause by reload

--- Additional Comments From pinskia at gcc dot gnu dot org 2004-10-25 19:14 --- The second patch has to due with "However, cleanup_subreg_operands first recognizes the insn, and this is where it presumably fails; due to this weird piece of code in general_operand:" -- http://gcc.g

[Bug target/15286] ICE cause by reload

--- Additional Comments From fjahanian at apple dot com 2004-10-25 19:12 --- You referred to them as 'both patches' in comment #21. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=15286

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-25 19:07 --- Subject: Re: ICE cause by reload > I applied the last two patch, but it didn;t help: The simplify-rtx.c patch is not *two* patches. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=15286

[Bug target/15286] ICE cause by reload

--- Additional Comments From fjahanian at apple dot com 2004-10-25 18:39 --- I applied the last two patch, but it didn;t help: % mygccf -O2 -mcpu=G5 -c loader_obj.i loader_obj.c: In function 'load_obj': loader_obj.c:92: error: unrecognizable insn: (insn 1395 601 1396 50 (set (subreg:DI (

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at gcc dot gnu dot org 2004-10-25 18:33 --- Correction. While the reload changes fix the crash, it looks like there still is a bug because the resulting code does not access the correct SUBREG. It does not access the LSW. -- http://gcc.gnu.org/bugz

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-25 18:04 --- Subject: Re: ICE cause by reload With both patches, the testcase works. This probably is a correct fix for reload. The code is a little messy and is cleaned up by implementing fixuns_trun

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-25 17:17 --- Well, as I understand it, SUBREG (MEM) is *supposed* to be generated for the case of SUBREG (pseudo) where the pseudo gets a stack slot. The scan_paradoxical_subregs mechanism was supposed to have allocate

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-25 16:47 --- Subject: Re: ICE cause by reload > uweigand at gcc dot gnu dot org writes: Ulrich> Does this patch help? It changes the error, but I'm not sure if it helps. pr15286.c: In function 'GMRESSolv

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-25 16:20 --- Does this patch help? Index: gcc/simplify-rtx.c === RCS file: /cvs/gcc/gcc/gcc/simplify-rtx.c,v retrieving revision 1.206 diff -c -p -r1.206

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-25 15:54 --- David Edelsohn wrote: >One could view this problem as reload not obeying CANNOT_CHANGE_MODE_CLASS >because it should not try to spill just the SImode SUBREG instead of creating a >DImode temporary on the s

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at gcc dot gnu dot org 2004-10-25 15:46 --- GCC is using fix_truncdfdi2 because no unsfix_truncdfsi2 pattern is defined and clock_t is an unsigned long. A short-term workaround is to implement an unsfix_truncdfsi2 pattern for PPC64 that uses fctid and exp

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at gcc dot gnu dot org 2004-10-24 22:52 --- This problem appears to be a confluence of issues between fix_truncdfdi2 and limitations of reload, exacerbated by Apple's LL64 mode. The problematic code is typedef unsigned long clock_t; clock_t clock(void);

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at gcc dot gnu dot org 2004-10-24 02:59 --- pr15286.c:72: error: unrecognizable insn: (insn 496 167 381 0 (set (mem:SI (plus:SI (reg/f:SI 1 r1) (const_int 204 [0xcc])) [15 clock_start+0 S4 A8]) (subreg:SI (reg:DI 32 f0) 4)) -1 (nil)

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-18 12:47 --- On s390, CANNOT_CHANGE_MODE_CLASS does indeed return false for such superclasses: #define CANNOT_CHANGE_MODE_CLASS(FROM, TO, CLASS) \ (GET_MODE_SIZE (FROM) != GET_MODE_SIZE (TO)

[Bug target/15286] ICE cause by reload

--- Additional Comments From dje at watson dot ibm dot com 2004-10-18 01:50 --- Subject: Re: ICE cause by reload (subreg:SI (reg:DI)) normally isn't a problem, except when reg:DI is assigned to an FPR. If reg:DI was assigned to an FPR, CLASS probably is NON_SPECIAL_REGS (enco

[Bug target/15286] ICE cause by reload

--- Additional Comments From uweigand at gcc dot gnu dot org 2004-10-15 10:46 --- As I understand it, generation of subregs the hardware does not support should be prevented by the CANNOT_CHANGE_MODE_CLASS mechanism. Reload would reload the full inner reg into a register that allows the