Kenneth Graunke <kenn...@whitecape.org> writes: > Since condition_list is an ordered list, we can just use enumerate() > when walking through it to get (index, value) pairs, rather than storing > a second dictionary mapping items to their indices. When looking for an > existing entry, use list.index() to get the index of that item. > --- > src/glsl/nir/nir_algebraic.py | 15 +++++---------- > 1 file changed, 5 insertions(+), 10 deletions(-) > > Here's my review feedback for this patch. I haven't tested it, but it does > compile at least. With this squashed in, patches 1-2 of the miniseries are > > Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
Yeah, if we don't have many conditions, the extra python code is probably not worth it.
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
