This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Repost: RFA [4.1]: improvement to if-conversion and cross-jumping (PR20070)


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



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]