Re: [Mesa-dev] [PATCH 1/2] nir: fix if condition propagation for alu use

2018-10-31 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand Thanks for figuring this out. This probably explains some of the hurt I was seeing with my series as well. --Jason On Tue, Oct 30, 2018 at 9:17 PM Timothy Arceri wrote: > From: Timothy Arceri > > We need to update the cursor before we check if the alu use is > do

[Mesa-dev] [PATCH 1/2] nir: fix if condition propagation for alu use

2018-10-30 Thread Timothy Arceri
From: Timothy Arceri We need to update the cursor before we check if the alu use is dominated by the if condition. Previously we were checking if the current location of the alu instruction was dominated by the if condition which would miss some optimisation opportunities. Fixes: a3b4cb34589e ("