On 2 May 2012 14:20, Chris Forbes <span dir="ltr">&lt;<a href="mailto:chrisf@ijw.co.nz" target="_blank">chrisf@ijw.co.nz</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">+void<br>
+brw_blorp_params::exec(struct intel_context *intel) const<br>
+{<br>
+   switch (intel-&gt;gen) {<br>
+   case 6:<br>
+      gen6_blorp_exec(intel, this);<br>
+      break;<br>
+   case 7:<br>
+      gen7_blorp_exec(intel, this);<br>
+      break;<br>
+   default:<br>
+      /* BLORP is not supported before Gen7. */<br>
+      assert(false);<br>
+      break;<br>
+   }<br>
+}<br>
<br>
</div>Presumably the comment in the default case should be Gen6 ?<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Chris<br>
</font></span></blockquote></div><br>Whoops, yes.  Fixed.<br>