[Mesa-dev] [PATCH 1/5] i965: Add writes_accumulator flag

Matt Turner mattst88 at gmail.com
Fri Mar 28 14:14:25 PDT 2014


On Fri, Mar 28, 2014 at 6:28 AM, Juha-Pekka Heikkila
<juhapekka.heikkila at gmail.com> wrote:
> +      if (inst->writes_accumulator||inst->writes_flag())

Spaces around ||

>           inst->dst = dst_reg(retype(brw_null_reg(), inst->dst.type));
> -         break;
> -      default:
> -         if (inst->writes_flag()) {
> -            inst->dst = dst_reg(retype(brw_null_reg(), inst->dst.type));
> -         } else {
> -            inst->remove();
> -         }
> -      }
> +      else
> +         inst->remove();

And let's use braces on nested if statements.


More information about the mesa-dev mailing list