On Thu, Aug 18, 2011 at 2:38 PM, Eric Anholt <e...@anholt.net> wrote:
> +   bool progress = true;
> +   while (progress) {
> +      progress = false;
> +      progress = dead_code_eliminate() || progress;

|| progress is always false, though maybe it's just written like that
to match the style of the code.

I'd have just written this as

bool progress;
do {
    progress = dead_code_eliminate();
} while (progress)

Matt
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to