This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Repost: RFA [4.1]: improvement to if-conversion and cross-jumping (PR20070)
- From: Bernd Schmidt <bernds_cb1 at t-online dot de>
- To: Joern RENNECKE <joern dot rennecke at st dot com>
- Cc: Steven Bosscher <stevenb at suse dot de>, Richard Henderson <rth at redhat dot com>, gcc-patches at gcc dot gnu dot org, jh at suse dot cz
- Date: Tue, 06 Dec 2005 03:00:40 +0100
- Subject: Re: Repost: RFA [4.1]: improvement to if-conversion and cross-jumping (PR20070)
- References: <41E59432.7080504@st.com> <42CD71E0.3070804@st.com> <42D3B666.6050701@st.com> <200507121547.21305.stevenb@suse.de> <42D3E705.9030507@st.com> <42D7C264.6030507@st.com> <438C567A.7030104@t-online.de> <438CCE0F.6020209@st.com> <438DC224.8020300@t-online.de> <438E1734.3090502@st.com> <438E3361.6050403@t-online.de> <4390BC11.5080001@st.com> <43946353.6020506@t-online.de> <4394D00D.9040402@st.com>
Joern RENNECKE wrote:
Bernd Schmidt wrote:
> We can still be called for a PRE_MODIFY / POST_MODIFY.
So? These do not contain SETs or CLOBBERs. The comment near the MEM
handling seems confused, and I'm not convinced we're doing the right
thing for these RTL expressions.
Oops, I they way I remembered them they contained a SET. They actually
contain a SET_SRC and a SET_DEST, except that these macros won't allow
these codes.
Could we change them to accept these codes? Then I can add PRE_MODIFY
and POST_MODIFY as additional case labels besides the SET: label.
Not sure this is such a good idea since the "SET_DEST" is actually an
in-out operand. I'd rather add them near with code similar to the
P{RE,OST}_{IN,DE}C cases.
Bernd