From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bill Currie To: Michael Hayes Cc: Bill Currie , law@cygnus.com, egcs@cygnus.com Subject: Re: reload segfault Date: Sun, 31 Jan 1999 11:47:00 -0000 Message-id: <36B4B3F8.F138C6CF@tssc.co.nz> References: <715.917582172@hurl.cygnus.com> <36B176BE.40047244@taniwha.tssc.co.nz> <14001.40431.954164.955361@ongaonga.elec.canterbury.ac.nz> X-SW-Source: 1999-01/msg00458.html Michael Hayes wrote: > > Bill Currie writes: > > > If I change the #define to just return CLASS (rather than the function > > call), this particular problem dissapears, but in other code I get: > > > > (insn 2193 1790 2194 (set (reg:PSI 31 r31) > > (high:PSI (lo_sum:PSI (reg:PSI 27 r27) > > (symbol_ref:SI ("system_include_depth"))))) -1 (nil) > > (nil)) > > > > which is definitly bogus. > > > > I'm going to continue tweaking my preferred_reload_class function and > > see if I can get it to work. As it seems to only be DFmode regs that > > give me problems, I should get something soon (I hope). > > FYI, I had to tweak PREFERRED_RELOAD_CLASS for the C4x recently as > well to fix a problem I had with an optional inherited reload. > (I think this was triggered by Jim Wilson's 2 Dec patch). > I ended up just returning CLASS to solve the problem. I was returning NO_REGS when I really should have been returning CLASS (just the one line in the whole function was wrong). However, I think I may have just discovered the root cause of all my problems: Insufficient strict versions of various macros, and possibly some missing ones. I believe the original i860 work is quite old, so it probably pre-dates many macros, definitly many of the recent changes. Lots of bitrot I guess. I'm currently having problems with incorrect regclasses (more PREFERRED_RELOAD_CLASS bogosity?), such as int regs being assigned to fp operands, but that may be due to me not getting my regclass defines right (I added a new class). Bill -- Leave others their otherness