From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeffrey A Law To: Bernd Schmidt Cc: meissner@cygnus.com, toon@moene.indiv.nluug.nl, egcs@cygnus.com Subject: Re: Reload patch to improve 386 code Date: Fri, 04 Sep 1998 05:03:00 -0000 Message-id: <16875.904900103@hurl.cygnus.com> References: X-SW-Source: 1998-09/msg00164.html Another reply to an old message from Bernd... In message you write: > Then, there are some simplifications that could be done. I don't like the > inheritance code, find_equiv_reg and all that. IMHO reload shouldn't try to > be very clever about this sort of thing - the reload_cse_regs pass can be made > more clever. While reload_cse_regs can help this stuff, I'm not sure that it totally eliminates the need for the reload inheritance stuff. In fact, I'm sure Joern can show you lots of case where improving inheritance leads to better code. So, I'm not sure it's time to ditch the inheritance code yet. Given the structure of the locally spilling reload code, I do see how reload inheriting gets noticably more complicated. One thing we should try is a cook off between the locally spilling reload code (with inheritance disabled) and the existing reload code. If the locally spilling reloader generally wins, I'll support disabling inheritance to get the benefit of local spilling. Then we can go back later and try to make inheritance work with your reload code. [ Then again, if you've made inheritance work, then there's no need for the cook off :-) I guess I'll find out when I start actually looking at the code. ] jeff