Re: [PATCH] Fix crossjumping (PR rtl-optimization/58365)

2013-09-10 Thread Jakub Jelinek
On Tue, Sep 10, 2013 at 01:18:34PM +0200, Richard Biener wrote: > Ok. Thanks. > > BTW, I wonder about REG_ATTRS and REG_POINTER, REG_USERVAR_P, > > REG_FUNCTION_VALUE_P, is it fine to keep them as is > > Definitely REG_EXPR is used to check for signedness of a register, but I > suppose > that i

Re: [PATCH] Fix crossjumping (PR rtl-optimization/58365)

2013-09-10 Thread Richard Biener
On Tue, Sep 10, 2013 at 12:38 PM, Jakub Jelinek wrote: > Hi! > > If two instructions are rtl_equal_p, and cross-jumping decides to cross-jump > them together, merge_memattrs takes care of clearing various MEM_ATTRS > or setting alignment to minimum, size to maximum etc. > But, it didn't handle fla

[PATCH] Fix crossjumping (PR rtl-optimization/58365)

2013-09-10 Thread Jakub Jelinek
Hi! If two instructions are rtl_equal_p, and cross-jumping decides to cross-jump them together, merge_memattrs takes care of clearing various MEM_ATTRS or setting alignment to minimum, size to maximum etc. But, it didn't handle flags on the MEMs. The following patch fixes that. MEM_KEEP_ALIAS_SE