On Tue, May 24, 2016 at 8:42 AM,  <[email protected]> wrote:
> From: Mathias Fröhlich <[email protected]>
>
> Replaces a loop that iterates all lights and test
> which of them is enabled by a loop only iterating over
> the bits set in the enabled bitmask.

This takes the code from something very obvious and easy to follow to
something you'll have to think twice about the correctness of...

Since MAX_LIGHTS is 8, this seems a bit like a premature optimization
to me. Does this patch yield any measurable improvement in speed in
any real-world applications?
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to