[Mesa-dev] [PATCH 3/3] nir: simplify node matching code when lowering to SSA

Caio Marcelo de Oliveira Filho caio.oliveira at intel.com
Wed Apr 11 06:11:08 UTC 2018


On Tue, Apr 10, 2018 at 10:15:45PM -0700, Jason Ekstrand wrote:
> On Tue, Apr 10, 2018 at 12:52 PM, Caio Marcelo de Oliveira Filho <
> caio.oliveira at intel.com> wrote:
> 
> > The matching code doesn't make real use of the return value. The main
> > function return value is ignored, and while the worker function
> > propagate its return value, the actual callback never returns false.
> >
> 
> When I wrote this, I don't think I realized it was only going to have the
> one use.  I was mostly following the pattern we'd used for all the other
> foreach functions in NIR.  That said, I think this is better.

I guessed that was the case.


> When a block contains multiple lines, we use { } regardless of whether or
> not it's just a single statement.

Got it!

Will send v2 soon.


Thanks,
Caio


More information about the mesa-dev mailing list