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

[patch] Don't try to reload match_operator

2011-09-17 Thread Richard Earnshaw
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. This is needed for an upcoming patch I ha