Re: [patch] Don't try to reload match_operator

2011-10-10 Thread Hans-Peter Nilsson
On Mon, 19 Sep 2011, Ulrich Weigand wrote: > Richard Earnshaw wrote: > > On 19/09/11 15:14, Ulrich Weigand wrote: > > > So it seems to me that for match_operator operands, the > > > goal_alternative_win flag should always be true ... > > > > > > Can you find out why this isn't true in your case? >

Re: [patch] Don't try to reload match_operator

2011-09-19 Thread Richard Earnshaw
On 19/09/11 16:53, Ulrich Weigand wrote: > Richard Earnshaw wrote: >> On 19/09/11 15:14, Ulrich Weigand wrote: >>> So it seems to me that for match_operator operands, the >>> goal_alternative_win flag should always be true ... >>> >>> Can you find out why this isn't true in your case? >> >> Hmm, I

Re: [patch] Don't try to reload match_operator

2011-09-19 Thread Ulrich Weigand
Richard Earnshaw wrote: > On 19/09/11 15:14, Ulrich Weigand wrote: > > So it seems to me that for match_operator operands, the > > goal_alternative_win flag should always be true ... > > > > Can you find out why this isn't true in your case? > > Hmm, I think it must be because of this: > >

Re: [patch] Don't try to reload match_operator

2011-09-19 Thread Richard Earnshaw
On 19/09/11 15:14, Ulrich Weigand wrote: > Richard Earnshaw wrote: > >> This patch fixes a couple of cases where reload incorrectly thinks it >> should be messing around with the result of a match_operator operand. >> Given that it's never even tried to validate the contents of such an >> operand

Re: [patch] Don't try to reload match_operator

2011-09-19 Thread Ulrich Weigand
Richard Earnshaw wrote: > This patch fixes a couple of cases where reload incorrectly thinks it > should be messing around with the result of a match_operator operand. > Given that it's never even tried to validate the contents of such an > operand the changes it tries to make are bogus. I'm wond